Skip to content

Fix CI config visualization missing classes

Frédéric Caplette requested to merge fc-fix-yaml-viz-graph into master

What does this MR do?

Due to some migration of scss from the pipelines bundle, some classes were no longer applied and it broke the visualization style. This uses new gitlab utilities to re-apply those styles and avoid having scss bundle. Note that the border radius classes are a bit smaller in REM and this is intentional (the actual visuals are not changing however)

This is behind a disabled feature flag so no changelog required.

Screenshots (strongly suggested)

Before

Screen_Shot_2020-11-02_at_1.23.48_PM

After

Screen_Shot_2020-11-02_at_1.18.25_PM

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 Frédéric Caplette

Merge request reports