Audit CSS utils being used as selectors in SCSS
In !154442 (merged), we initially added a script that checked whether CSS utils were being used as selectors in GitLab's SCSS. We did not keep the script, but here are its latest findings:
- [-]
app/assets/stylesheets/components/whats_new.scss
includesgl-spin
: False positive (.gl-spinner-container
). - [-]
app/assets/stylesheets/framework/animations.scss
includesgl-spin
: False positive (.gl-spinner
). - [-]
app/assets/stylesheets/framework/buttons.scss
includesgl-spin
: False positive (.gl-spinner
). -
app/assets/stylesheets/framework/common.scss
includesgl-font-sm
: !158830 (merged). -
app/assets/stylesheets/framework/common.scss
includesgl-font-base
: !158830 (merged). -
app/assets/stylesheets/framework/common.scss
includesgl-font-lg
: !158830 (merged) - [-]
app/assets/stylesheets/framework/common.scss
includesgl-line-height-1
: False positive as it's actuallygl-line-height-14
, but it should be moved to Tailwind anyways -> gitlab-ui!4384 (closed). - [-]
app/assets/stylesheets/framework/dropdowns.scss
includesgl-spin
: False positive (.gl-spinner
) - [-]
app/assets/stylesheets/page_bundles/_pipeline_mixins.scss
includesgl-spin
: False positive (.gl-spinner
). -
app/assets/stylesheets/page_bundles/trial_discover_page.scss
includesgl-flex-basis-third
: !158830 (merged). -
app/assets/stylesheets/page_bundles/trial_discover_page.scss
includesgl-flex-basis-half
: !158830 (merged). -
app/assets/stylesheets/pages/commits.scss
includesgl-text-truncate
: !158830 (merged). -
app/assets/stylesheets/utilities.scss
includesgl-hover-border-gray-100
: !158830 (merged). -
app/assets/stylesheets/utilities.scss
includesgl-sm-mr-0
: !158830 (merged). -
ee/app/assets/stylesheets/components/ai_agent_chat.scss
includesgl-display-flex
: !158830 (merged).
We need to audit those findings to get rid of potential CSS utils selectors in the stylesheets.
Edited by Paul Gascou-Vaillancourt