Skip to content

Commit

Permalink
Remove width non-responsive override (#248)
Browse files Browse the repository at this point in the history
**Why**: These were kept in #236 as they differed from USWDS defaults, but they _were_ identical to the default as of the original file creation. USWDS later [changed the default](uswds/uswds@16806bd), and since we had no explicit reason to disable responsive styles, we should inherit the default instead.
  • Loading branch information
aduth authored Sep 13, 2021
1 parent ae873d7 commit 29d3b33
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 15 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

- All theme variables are now set as [default values](https://sass-lang.com/documentation/variables#default-values), allowing you to override them for per-project requirements.
- Note: Since the design system is meant to be an opinionated set of smart defaults, it's recommended to use restraint with variable customization, or at least consider when it may be more appropriate to adjust a setting from the design system itself in order to maintain consistency across projects.
- Responsive variants of width utility classes are now enabled.

## 6.1.0

Expand Down
15 changes: 0 additions & 15 deletions src/scss/uswds-theme/_utilities.scss
Original file line number Diff line number Diff line change
Expand Up @@ -31,18 +31,3 @@ $global-color-palettes: (
'palette-color-default',
'palette-color-state',
) !default;

/*
----------------------------------------
Settings
----------------------------------------
*/

$width-settings: (
output: true,
responsive: false,
active: false,
focus: false,
hover: false,
visited: false,
) !default;

0 comments on commit 29d3b33

Please sign in to comment.