Skip to content

Remove many unused bg- CSS classes

Simon Knox requested to merge psi-bg-css into master

What does this MR do and why?

Removes 6kB from application_utilities.css. We were adding all variations of bg-success-50 -> bg-success-900 for all variants, but most were unused.

Screenshots or screen recordings

No visual changes.

How to set up and validate locally

Search for usages for any bg-* classes I missed. I used this regex: (?!gl-)bg-(primary|secondary|success|warning|danger)- The trailing hyphen is because bg-secondary still works (that comes from bootstrap).

MR acceptance checklist

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

Edited by Simon Knox

Merge request reports