Skip to content

Remove ci-status-link class

Payton Burdette requested to merge 418753-ci-status-class-port into master

What does this MR do and why?

Addresses the rest of the issue #418753 (closed) by removing ci-status-link class references and ensuring styles do not break.

First part: !133369 (merged)

Screenshots or screen recordings

These screenshots are the result of removing the ci-status-link class

_tag.html.haml

Screenshot 2023-10-16 at 4.23.45 PM.png

last_commit.vue

Screenshot 2023-10-16 at 2.34.34 PM.png

_branch.html.haml

Screenshot 2023-10-16 at 4.21.36 PM.png

_project.html.haml

Screenshot 2023-10-16 at 4.28.22 PM.png

_merge_request.html.haml

Screenshot 2023-10-16 at 4.30.47 PM.png

_merge_request_pipeline_status.html.haml

Screenshot 2023-10-16 at 4.36.08 PM.png

_related_branches.html.haml

Screenshot 2023-10-17 at 10.13.08 AM.png

How to set up and validate locally

Easiest way to test this MR out is to just visit these locations on prod, inspect the element and remove the class ci-status-link and verify everything is working. If CSS changes were added in replacement then checkout the branch and visit those locations. Or add CSS classes manually in browser.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #418753 (closed)

Edited by Payton Burdette

Merge request reports