Skip to content

Commit 4f8f4aa

Browse files
fix(swatch): revert all themes to match original spec
1 parent 8ea01bd commit 4f8f4aa

File tree

5 files changed

+10
-17
lines changed

5 files changed

+10
-17
lines changed

.changeset/fast-hats-worry.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
"@spectrum-css/swatch": patch
2+
"@spectrum-css/swatch": minor
33
---
44

5-
Revert S1/Express disabled icon color back to original spectrum-white.
5+
Reverts all themes' disabled icon color back to original spectrum-white and border color to black with 51% opacity. Styling for path:first-child and path:last-child have been removed because the icon only has one single path, stroke and fill are used to style the icon color and outline instead.

components/swatch/dist/metadata.json

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@
2626
".spectrum-Swatch--sizeS",
2727
".spectrum-Swatch--sizeXS",
2828
".spectrum-Swatch-disabledIcon",
29-
".spectrum-Swatch-disabledIcon path:first-child",
30-
".spectrum-Swatch-disabledIcon path:last-child",
3129
".spectrum-Swatch-fill",
3230
".spectrum-Swatch-fill:before",
3331
".spectrum-Swatch-image",
@@ -79,6 +77,8 @@
7977
"--spectrum-swatch-border-thickness",
8078
"--spectrum-swatch-border-thickness-selected",
8179
"--spectrum-swatch-dash-icon-color",
80+
"--spectrum-swatch-disabled-icon-border-color",
81+
"--spectrum-swatch-disabled-icon-border-opacity",
8282
"--spectrum-swatch-disabled-icon-color",
8383
"--spectrum-swatch-disabled-icon-size",
8484
"--spectrum-swatch-focus-indicator-border-radius",
@@ -101,6 +101,7 @@
101101
],
102102
"global": [
103103
"--spectrum-animation-duration-100",
104+
"--spectrum-black-rgb",
104105
"--spectrum-border-width-100",
105106
"--spectrum-border-width-200",
106107
"--spectrum-corner-radius-75",
@@ -113,6 +114,7 @@
113114
"--spectrum-gray-900",
114115
"--spectrum-picked-color",
115116
"--spectrum-red-900",
117+
"--spectrum-white",
116118
"--spectrum-workflow-icon-size-100",
117119
"--spectrum-workflow-icon-size-200",
118120
"--spectrum-workflow-icon-size-50",
@@ -121,7 +123,6 @@
121123
"system-theme": [
122124
"--system-swatch-border-color",
123125
"--system-swatch-border-radius",
124-
"--system-swatch-disabled-icon-color",
125126
"--system-swatch-inner-border-color-selected"
126127
],
127128
"passthroughs": [],

components/swatch/index.css

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,8 @@
6161

6262
.spectrum-Swatch {
6363
--spectrum-swatch-focus-indicator-border-radius: 8px;
64-
--spectrum-swatch-icon-border-color: rgb(0 0 0 / 51%);
64+
--spectrum-swatch-icon-border-color: rgba(var(--spectrum-black-rgb), var(--spectrum-swatch-disabled-icon-border-opacity));
65+
--spectrum-swatch-disabled-icon-color: var(--spectrum-white);
6566
--spectrum-swatch-border-thickness: var(--spectrum-border-width-100);
6667
--spectrum-swatch-border-thickness-selected: var(--spectrum-border-width-200);
6768
--spectrum-swatch-focus-indicator-thickness: var(--spectrum-focus-indicator-thickness);
@@ -265,18 +266,11 @@
265266
display: none;
266267
pointer-events: none;
267268

268-
color: var(--highcontrast-swatch-disabled-icon-color, var(--mod-swatch-disabled-icon-color, var(--spectrum-swatch-disabled-icon-color)));
269-
stroke: var(--highcontrast-swatch-disabled-icon-color, var(--mod-swatch-disabled-icon-color, var(--spectrum-swatch-disabled-icon-color)));
270-
271269
/* Icon fill color */
272-
path:first-child {
273-
fill: var(--highcontrast-swatch-disabled-icon-color, var(--mod-swatch-disabled-icon-color, var(--spectrum-swatch-disabled-icon-color)));
274-
}
270+
color: var(--highcontrast-swatch-disabled-icon-color, var(--mod-swatch-disabled-icon-color, var(--spectrum-swatch-disabled-icon-color)));
275271

276272
/* Icon outline color - does not need a highcontrast token because the icon fill color provides contrast. */
277-
path:last-child {
278-
fill: var(--mod-swatch-icon-border-color, var(--spectrum-swatch-icon-border-color));
279-
}
273+
stroke: var(--mod-swatch-icon-border-color, var(--spectrum-swatch-icon-border-color));
280274
}
281275

282276
.spectrum-Swatch--rectangle {

components/swatch/themes/spectrum-two.css

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
.spectrum-Swatch {
1717
--spectrum-swatch-border-color: rgba(var(--spectrum-gray-1000-rgb), var(--spectrum-swatch-border-color-opacity));
1818
--spectrum-swatch-inner-border-color-selected: var(--spectrum-gray-25);
19-
--spectrum-swatch-disabled-icon-color: var(--spectrum-gray-25);
2019
--spectrum-swatch-border-radius: var(--spectrum-corner-radius-75);
2120
}
2221
}

components/swatch/themes/spectrum.css

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
.spectrum-Swatch {
2020
--spectrum-swatch-border-color: rgba(var(--spectrum-gray-900-rgb), var(--spectrum-swatch-border-color-opacity));
2121
--spectrum-swatch-inner-border-color-selected: var(--spectrum-gray-50);
22-
--spectrum-swatch-disabled-icon-color: var(--spectrum-white);
2322
--spectrum-swatch-border-radius: var(--spectrum-corner-radius-100);
2423
}
2524
}

0 commit comments

Comments
 (0)