Skip to content

UX polish for pipeline status in pipeline editor

Mireya Andres requested to merge pipeline-status-ux into master

What does this MR do?

A couple of UX polishing from this discussion:

  • When there's a commit confirmation alert, there's no space between the status area and the alert. I think we could use 16px distance there.
  • The pipeline status icon should be vertically centered related to the text, now it's a bit higher than text

Testing Locally

  • Enable the :pipeline_status_for_pipeline_editor feature flag.
  • Go to the Pipeline Editor (CI/CD > Editor), make some changes, and commit those changes.
  • Verify changes.

Screenshots

Before After
Before After

Before and After Commit

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Mireya Andres

Merge request reports