Skip to content

Commit 79e3363

Browse files
authored
fix(button): reduce mod interception (#3662)
This update aims to simplify `--mod-*` access by ensuring local variants and states aren't hooking into those custom properties for overrides. This updates all local variants and states to override the `--spectrum-button-*` properties instead and adjusts the specificity to ensure no regressions in rendered results. Other changes of note: - Removes the `.spectrum-Button--sizeM` as unnecessary and duplicate as these styles are applied in the base class. - Reduces the use of the `:not()` pseudo-class in theme overrides as these selectors are more complex to remap for web component projects.
1 parent 415952b commit 79e3363

File tree

6 files changed

+444
-448
lines changed

6 files changed

+444
-448
lines changed

.changeset/stale-baboons-shake.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
---
2+
"@spectrum-css/button": patch
3+
---
4+
5+
This update aims to simplify `--mod-*` access by ensuring local variants and states aren't hooking into those custom properties for overrides. This updates all local variants and states to override the `--spectrum-button-*` properties instead and adjusts the specificity to ensure no regressions in rendered results.
6+
7+
Other changes of note:
8+
9+
- Removes the `.spectrum-Button--sizeM` as unnecessary and duplicate as these styles are applied in the base class.
10+
- Reduces the use of the `:not()` pseudo-class in theme overrides as these selectors are more complex to remap for web component projects.

components/button/dist/metadata.json

Lines changed: 35 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
".spectrum-Button .spectrum-ProgressCircle",
99
".spectrum-Button--noWrap .spectrum-Button-label",
1010
".spectrum-Button--sizeL",
11-
".spectrum-Button--sizeM",
1211
".spectrum-Button--sizeS",
1312
".spectrum-Button--sizeXL",
1413
".spectrum-Button-label",
@@ -35,26 +34,19 @@
3534
".spectrum-Button.spectrum-Button--quiet",
3635
".spectrum-Button.spectrum-Button--secondary",
3736
".spectrum-Button.spectrum-Button--secondary.spectrum-Button--outline",
38-
".spectrum-Button.spectrum-Button--secondary:not(.spectrum-Button--outline)",
3937
".spectrum-Button.spectrum-Button--staticBlack",
4038
".spectrum-Button.spectrum-Button--staticBlack.is-selected",
4139
".spectrum-Button.spectrum-Button--staticBlack.spectrum-Button--outline",
42-
".spectrum-Button.spectrum-Button--staticBlack.spectrum-Button--outline.spectrum-Button--secondary",
43-
".spectrum-Button.spectrum-Button--staticBlack.spectrum-Button--outline:not(.spectrum-Button--secondary)",
4440
".spectrum-Button.spectrum-Button--staticBlack.spectrum-Button--quiet",
4541
".spectrum-Button.spectrum-Button--staticBlack.spectrum-Button--secondary",
4642
".spectrum-Button.spectrum-Button--staticBlack.spectrum-Button--secondary.spectrum-Button--outline",
47-
".spectrum-Button.spectrum-Button--staticBlack.spectrum-Button--secondary:not(.spectrum-Button--outline)",
4843
".spectrum-Button.spectrum-Button--staticWhite",
4944
".spectrum-Button.spectrum-Button--staticWhite.is-selected",
5045
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--accent",
5146
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--outline",
52-
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--outline.spectrum-Button--secondary",
53-
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--outline:not(.spectrum-Button--secondary)",
5447
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--quiet",
5548
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--secondary",
5649
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--secondary.spectrum-Button--outline",
57-
".spectrum-Button.spectrum-Button--staticWhite.spectrum-Button--secondary:not(.spectrum-Button--outline)",
5850
".spectrum-Button::-moz-focus-inner",
5951
".spectrum-Button:active",
6052
".spectrum-Button:after",
@@ -311,9 +303,9 @@
311303
"--system-button-primary-outline-background-color-focus",
312304
"--system-button-primary-outline-background-color-hover",
313305
"--system-button-secondary-background-color-default",
314-
"--system-button-secondary-not-outline-background-color-down",
315-
"--system-button-secondary-not-outline-background-color-focus",
316-
"--system-button-secondary-not-outline-background-color-hover",
306+
"--system-button-secondary-background-color-down",
307+
"--system-button-secondary-background-color-focus",
308+
"--system-button-secondary-background-color-hover",
317309
"--system-button-secondary-outline-background-color-down",
318310
"--system-button-secondary-outline-background-color-focus",
319311
"--system-button-secondary-outline-background-color-hover",
@@ -331,21 +323,18 @@
331323
"--system-button-static-black-content-color-down",
332324
"--system-button-static-black-content-color-focus",
333325
"--system-button-static-black-content-color-hover",
334-
"--system-button-static-black-outline-not-secondary-background-color-default",
335-
"--system-button-static-black-outline-not-secondary-background-color-down",
336-
"--system-button-static-black-outline-not-secondary-background-color-focus",
337-
"--system-button-static-black-outline-not-secondary-background-color-hover",
338-
"--system-button-static-black-outline-not-secondary-border-color-default",
339-
"--system-button-static-black-outline-not-secondary-border-color-down",
340-
"--system-button-static-black-outline-not-secondary-border-color-focus",
341-
"--system-button-static-black-outline-not-secondary-border-color-hover",
342-
"--system-button-static-black-outline-not-secondary-content-color-default",
343-
"--system-button-static-black-outline-not-secondary-content-color-down",
344-
"--system-button-static-black-outline-not-secondary-content-color-focus",
345-
"--system-button-static-black-outline-not-secondary-content-color-hover",
346-
"--system-button-static-black-outline-secondary-background-color-down",
347-
"--system-button-static-black-outline-secondary-background-color-focus",
348-
"--system-button-static-black-outline-secondary-background-color-hover",
326+
"--system-button-static-black-outline-background-color-default",
327+
"--system-button-static-black-outline-background-color-down",
328+
"--system-button-static-black-outline-background-color-focus",
329+
"--system-button-static-black-outline-background-color-hover",
330+
"--system-button-static-black-outline-border-color-default",
331+
"--system-button-static-black-outline-border-color-down",
332+
"--system-button-static-black-outline-border-color-focus",
333+
"--system-button-static-black-outline-border-color-hover",
334+
"--system-button-static-black-outline-content-color-default",
335+
"--system-button-static-black-outline-content-color-down",
336+
"--system-button-static-black-outline-content-color-focus",
337+
"--system-button-static-black-outline-content-color-hover",
349338
"--system-button-static-black-secondary-background-color-default",
350339
"--system-button-static-black-secondary-background-color-down",
351340
"--system-button-static-black-secondary-background-color-focus",
@@ -354,6 +343,10 @@
354343
"--system-button-static-black-secondary-content-color-down",
355344
"--system-button-static-black-secondary-content-color-focus",
356345
"--system-button-static-black-secondary-content-color-hover",
346+
"--system-button-static-black-secondary-outline-background-color-default",
347+
"--system-button-static-black-secondary-outline-background-color-down",
348+
"--system-button-static-black-secondary-outline-background-color-focus",
349+
"--system-button-static-black-secondary-outline-background-color-hover",
357350
"--system-button-static-black-secondary-outline-border-color-default",
358351
"--system-button-static-black-secondary-outline-border-color-down",
359352
"--system-button-static-black-secondary-outline-border-color-focus",
@@ -366,21 +359,18 @@
366359
"--system-button-static-white-content-color-down",
367360
"--system-button-static-white-content-color-focus",
368361
"--system-button-static-white-content-color-hover",
369-
"--system-button-static-white-outline-not-secondary-background-color-default",
370-
"--system-button-static-white-outline-not-secondary-background-color-down",
371-
"--system-button-static-white-outline-not-secondary-background-color-focus",
372-
"--system-button-static-white-outline-not-secondary-background-color-hover",
373-
"--system-button-static-white-outline-not-secondary-border-color-default",
374-
"--system-button-static-white-outline-not-secondary-border-color-down",
375-
"--system-button-static-white-outline-not-secondary-border-color-focus",
376-
"--system-button-static-white-outline-not-secondary-border-color-hover",
377-
"--system-button-static-white-outline-not-secondary-content-color-default",
378-
"--system-button-static-white-outline-not-secondary-content-color-down",
379-
"--system-button-static-white-outline-not-secondary-content-color-focus",
380-
"--system-button-static-white-outline-not-secondary-content-color-hover",
381-
"--system-button-static-white-outline-secondary-background-color-down",
382-
"--system-button-static-white-outline-secondary-background-color-focus",
383-
"--system-button-static-white-outline-secondary-background-color-hover",
362+
"--system-button-static-white-outline-background-color-default",
363+
"--system-button-static-white-outline-background-color-down",
364+
"--system-button-static-white-outline-background-color-focus",
365+
"--system-button-static-white-outline-background-color-hover",
366+
"--system-button-static-white-outline-border-color-default",
367+
"--system-button-static-white-outline-border-color-down",
368+
"--system-button-static-white-outline-border-color-focus",
369+
"--system-button-static-white-outline-border-color-hover",
370+
"--system-button-static-white-outline-content-color-default",
371+
"--system-button-static-white-outline-content-color-down",
372+
"--system-button-static-white-outline-content-color-focus",
373+
"--system-button-static-white-outline-content-color-hover",
384374
"--system-button-static-white-secondary-background-color-default",
385375
"--system-button-static-white-secondary-background-color-down",
386376
"--system-button-static-white-secondary-background-color-focus",
@@ -389,6 +379,10 @@
389379
"--system-button-static-white-secondary-content-color-down",
390380
"--system-button-static-white-secondary-content-color-focus",
391381
"--system-button-static-white-secondary-content-color-hover",
382+
"--system-button-static-white-secondary-outline-background-color-default",
383+
"--system-button-static-white-secondary-outline-background-color-down",
384+
"--system-button-static-white-secondary-outline-background-color-focus",
385+
"--system-button-static-white-secondary-outline-background-color-hover",
392386
"--system-button-static-white-secondary-outline-border-color-default",
393387
"--system-button-static-white-secondary-outline-border-color-down",
394388
"--system-button-static-white-secondary-outline-border-color-focus",

0 commit comments

Comments
 (0)