Skip to content

Commit

Permalink
fix: specificity adjustments for variant application via theming
Browse files Browse the repository at this point in the history
  • Loading branch information
castastrophe committed Nov 13, 2024
1 parent bd08d68 commit 91e33e0
Show file tree
Hide file tree
Showing 57 changed files with 969 additions and 1,033 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
dist
!tokens/dist/index.css
!tokens/dist/css/*.css

# Not committing the map assets, these are dev-only
*.map
Expand Down
8 changes: 4 additions & 4 deletions components/actionbutton/stories/actionbutton.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -63,12 +63,12 @@ export const ActionButtonGroup = Variants({
staticColor: "black",
},
{
testHeading: "Static black - emphasized",
testHeading: "Static black (emphasized)",
staticColor: "black",
isEmphasized: true,
},
{
testHeading: "Static black - quiet",
testHeading: "Static black (quiet)",
staticColor: "black",
isQuiet: true,
},
Expand All @@ -77,12 +77,12 @@ export const ActionButtonGroup = Variants({
staticColor: "white",
},
{
testHeading: "Static white - emphasized",
testHeading: "Static white (emphasized)",
staticColor: "white",
isEmphasized: true,
},
{
testHeading: "Static white - quiet",
testHeading: "Static white (quiet)",
staticColor: "white",
isQuiet: true,
},
Expand Down
4 changes: 3 additions & 1 deletion components/avatar/metadata/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
"sourceFile": "index.css",
"selectors": [
".spectrum-Avatar",
".spectrum-Avatar .spectrum-Avatar.spectrum-Avatar--size100",
".spectrum-Avatar-image",
".spectrum-Avatar-link",
".spectrum-Avatar.is-disabled",
".spectrum-Avatar.is-focused:not(.is-disabled):after",
".spectrum-Avatar.spectrum-Avatar--size100",
".spectrum-Avatar.spectrum-Avatar--size200",
".spectrum-Avatar.spectrum-Avatar--size300",
".spectrum-Avatar.spectrum-Avatar--size400",
Expand Down Expand Up @@ -53,11 +53,13 @@
"--spectrum-focus-indicator-thickness"
],
"system-theme": [
"--system-avatar-block-size",
"--system-avatar-color-opacity",
"--system-avatar-color-opacity-disabled",
"--system-avatar-focus-indicator-color",
"--system-avatar-focus-indicator-gap",
"--system-avatar-focus-indicator-thickness",
"--system-avatar-inline-size",
"--system-avatar-size-100-block-size",
"--system-avatar-size-100-inline-size",
"--system-avatar-size-200-block-size",
Expand Down
2 changes: 1 addition & 1 deletion components/avatar/themes/spectrum-two.css
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@

--spectrum-avatar-color-opacity-disabled: var(--spectrum-avatar-opacity-disabled);

&
&,
&.spectrum-Avatar--size100 {
--spectrum-avatar-inline-size: var(--spectrum-avatar-size-100);
--spectrum-avatar-block-size: var(--spectrum-avatar-size-100);
Expand Down
9 changes: 4 additions & 5 deletions components/button/index.css
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@
.spectrum-Icon {
/* Any block-size difference between the intended workflow icon size and actual icon used.
Helps support any existing use of smaller UI icons instead of intended Workflow icons. */
/* stylelint-disable-next-line spectrum-tools/no-unknown-custom-properties -- use the icon sizing to determine the size calculation */
--spectrum-button-icon-size-difference: max(0px, var(--spectrum-button-intended-icon-size) - var(--spectrum-icon-size, var(--spectrum-button-intended-icon-size)));

margin-block-start: var(--mod-button-icon-margin-block-start, max(0px, var(--mod-button-top-to-icon, var(--spectrum-button-top-to-icon)) - var(--mod-button-border-width, var(--spectrum-button-border-width)) + (var(--spectrum-button-icon-size-difference, 0px) / 2)));
Expand Down Expand Up @@ -201,10 +200,10 @@ a.spectrum-Button {
--spectrum-button-focus-indicator-color: var(--mod-static-black-focus-indicator-color, var(--spectrum-static-black-focus-indicator-color));

&.is-selected {
--spectrum-button-content-color-default: var(--mod-button-static-content-color, var(--spectrum-button-static-white-content-color-default));
--spectrum-button-content-color-hover: var(--mod-button-static-content-color, var(--spectrum-button-static-white-content-color-hover));
--spectrum-button-content-color-down: var(--mod-button-static-content-color, var(--spectrum-button-static-white-content-color-down));
--spectrum-button-content-color-focus: var(--mod-button-static-content-color, var(--spectrum-button-static-white-content-color-focus));
--mod-button-content-color-default: var(--mod-button-static-content-color);
--mod-button-content-color-hover: var(--mod-button-static-content-color);
--mod-button-content-color-down: var(--mod-button-static-content-color);
--mod-button-content-color-focus: var(--mod-button-static-content-color);
}
}

Expand Down
8 changes: 2 additions & 6 deletions components/button/metadata/metadata.json
Original file line number Diff line number Diff line change
Expand Up @@ -154,10 +154,6 @@
"--spectrum-button-min-width",
"--spectrum-button-minimum-width-multiplier",
"--spectrum-button-padding-label-to-icon",
"--spectrum-button-static-white-content-color-default",
"--spectrum-button-static-white-content-color-down",
"--spectrum-button-static-white-content-color-focus",
"--spectrum-button-static-white-content-color-hover",
"--spectrum-button-top-to-icon",
"--spectrum-button-top-to-text",
"--spectrum-button-top-to-text-extra-large",
Expand Down Expand Up @@ -525,7 +521,7 @@
"--system-button-static-black-content-color-down",
"--system-button-static-black-content-color-focus",
"--system-button-static-black-content-color-hover",
"--system-button-static-black-focus-indicator-color",
"--system-button-static-black-focus-ring-color",
"--system-button-static-black-outline-background-color-default",
"--system-button-static-black-outline-background-color-disabled",
"--system-button-static-black-outline-background-color-down",
Expand Down Expand Up @@ -594,7 +590,7 @@
"--system-button-static-white-content-color-down",
"--system-button-static-white-content-color-focus",
"--system-button-static-white-content-color-hover",
"--system-button-static-white-focus-indicator-color",
"--system-button-static-white-focus-ring-color",
"--system-button-static-white-outline-background-color-default",
"--system-button-static-white-outline-background-color-disabled",
"--system-button-static-white-outline-background-color-down",
Expand Down
Loading

0 comments on commit 91e33e0

Please sign in to comment.