Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused CSS utility classes #5401

Merged
merged 1 commit into from Sep 14, 2021
Merged

Remove unused CSS utility classes #5401

merged 1 commit into from Sep 14, 2021

Conversation

aduth
Copy link
Member

@aduth aduth commented Sep 14, 2021

Why:

  • Moves us toward design system utilities, removing temptation of use for developers and facilitating future substitution work.
  • Reduces size of stylesheets, producing smaller bundle sizes for users (faster page loads) and lower mental overhead for developers.

**Why**:

- Moves us toward design system utilities, removing temptation of use for developers and facilitating future substitution work.
- Reduces size of stylesheets, producing smaller bundle sizes for users (faster page loads) and lower mental overhead for developers.
.sm-mtn3 { margin-top: -$space-3; }
}

.margin-top-neg-4 {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of these design-system extensions are still in use, but they've been made available directly through newer releases of USWDS.

https://github.com/uswds/uswds/releases/tag/v2.12.0

Added more negative margin tokens to margin utilities. Now you can use utilities like margin-neg-4 and margin-neg-5. (uswds/uswds#4212)

See source: https://unpkg.com/uswds@2.12.1/dist/css/uswds.css

We inherit USWDS 2.12.1 as of #5383.

@aduth aduth merged commit 64e8826 into main Sep 14, 2021
@aduth aduth deleted the aduth-rm-unused-css-util branch September 14, 2021 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants