Skip to content

Commit e4556f1

Browse files
authored
feat(checkbox/group): s2 token migration, control spacing, subtle color changes (#3531)
* feat(checkbox): adding new s2 tokens * chore(checkbox): adding necessary files * chore(checkbox): adding necessary files and dist files * chore(checkbox): making tokens more specific * chore(checkbox): adding checkbox dist files * chore(checkbox): remove unnecessary dist files * feat(checkbox): adding checkbox s2 tokens * chore(checkbox): adding metadata file * feat(checkbox): restoring new s2 tokens * chore(checkbox): adding metadata file * feat(checkbox): adding focused and hovered states in test env * feat(checkbox): configure WHCM mode and modify css * chore(checkbox): adding metadata file * feat(checkbox): updating changeset, removing some conflicting classes * chore(checkbox): restoring s2 files * feat(checkbox): fixing emphasized focused state * chore(checkbox): adding metadata * feat(checkbox): updating whcm hover states * feat(checkbox): updating whcm * feat(checkbox): bring back size M * feat(checkbox): updating WHCM and metadata * feat(checkbox): updating checkbox states * feat(checkbox): adding new s2 tokens * chore(checkbox): adding necessary files and dist files * chore(checkbox): making tokens more specific * chore(checkbox): adding checkbox dist files * chore(checkbox): remove unnecessary dist files * feat(checkbox): adding checkbox s2 tokens * feat(checkbox): restoring new s2 tokens * feat(checkbox): adding focused and hovered states in test env * feat(checkbox): configure WHCM mode and modify css * feat(checkbox): updating changeset, removing some conflicting classes * feat(checkbox): fixing emphasized focused state * chore(checkbox): adding metadata * feat(checkbox): updating whcm hover states * feat(checkbox): updating whcm * feat(checkbox): bring back size M * feat(checkbox): updating WHCM and metadata * feat(checkbox): updating checkbox states * chore(checkbox): restoring metadata files * feat(checkbox): fix emphasized unchecked focus state * feat(checkbox): update emphasized error state * chore(checkbox): restore metadata files * feat(checkbox): removing is-focus-visible and is-disabled class usage * chore(checkbox): removing metadata files * feat(checkbox): cleaned house - reorg classes, removed bad selects, cut out some tests * chore(checkbox): restoring search metadata * feat(checkbox): adding active to test grid * feat(checkbox): adding checkbox downstate fix, removing is-active class, and finding indeterminate bug * feat(checkbox): add back test cases * feat(checkbox): adding hovered invalid * feat(checkbox): disable all hover states in disabled state
1 parent 888e63d commit e4556f1

File tree

6 files changed

+976
-459
lines changed

6 files changed

+976
-459
lines changed

.changeset/six-donuts-march.md

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
---
2+
"@spectrum-css/checkbox": major
3+
---
4+
5+
#### Checkbox S2 Migration
6+
7+
Checkbox's control and color tokens have been updated by the latest S2 tokens:
8+
9+
**Updates**
10+
11+
- Checkbox has a background color token: `--spectrum-checkbox-background-color`
12+
- New typography additions: line-height, font-weight, and font style
13+
- `spectrum-Checkbox--sizeM` is now the default style settings of the component.
14+
- Fixed typo in `--mod-focus-indicator-thickness`
15+
- Removed interactive state selectors in `--emphasized` variant that was overriding other variants
16+
- Refactored WHCM settings for read-only and disabled states
17+
- `--mod-checkbox-spacing` has been renamed to `--mod-checkbox-margin-block`
18+
19+
##### New tokens
20+
21+
`--spectrum-component-size-width-ratio-down`
22+
`--spectrum-checkbox-bottom-to-text`
23+
`--spectrum-checkbox-top-to-control`
24+
`--spectrum-accent-content-color-default`
25+
`--spectrum-accent-content-color-hover`
26+
`--spectrum-accent-content-color-down`
27+
`--spectrum-accent-content-color-key-focus`
28+
29+
##### Modified tokens
30+
31+
`--spectrum-checkbox-checkmark-color`
32+
`--spectrum-checkbox-invalid-color-down`
33+
`--spectrum-checkbox-control-color-default`
34+
`--spectrum-checkbox-control-color-hover`
35+
`--spectrum-checkbox-control-color-down`
36+
`--spectrum-checkbox-control-color-focus`

components/checkbox/dist/metadata.json

Lines changed: 50 additions & 76 deletions
Original file line numberDiff line numberDiff line change
@@ -2,86 +2,64 @@
22
"sourceFile": "index.css",
33
"selectors": [
44
".spectrum-Checkbox",
5+
".spectrum-Checkbox .spectrum-Checkbox-box",
6+
".spectrum-Checkbox .spectrum-Checkbox-box:after",
7+
".spectrum-Checkbox .spectrum-Checkbox-box:before",
58
".spectrum-Checkbox .spectrum-Checkbox-checkmark",
9+
".spectrum-Checkbox .spectrum-Checkbox-input",
10+
".spectrum-Checkbox .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box .spectrum-Checkbox-checkmark",
11+
".spectrum-Checkbox .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
612
".spectrum-Checkbox .spectrum-Checkbox-input:checked:disabled + .spectrum-Checkbox-box:before",
713
".spectrum-Checkbox .spectrum-Checkbox-input:checked:disabled ~ .spectrum-Checkbox-label",
14+
".spectrum-Checkbox .spectrum-Checkbox-input:checked:hover:disabled + .spectrum-Checkbox-box:before",
15+
".spectrum-Checkbox .spectrum-Checkbox-input:disabled",
816
".spectrum-Checkbox .spectrum-Checkbox-input:disabled + .spectrum-Checkbox-box:before",
917
".spectrum-Checkbox .spectrum-Checkbox-input:disabled ~ .spectrum-Checkbox-label",
18+
".spectrum-Checkbox .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:after",
19+
".spectrum-Checkbox .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:before",
20+
".spectrum-Checkbox .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-label",
21+
".spectrum-Checkbox .spectrum-Checkbox-input:hover:disabled + .spectrum-Checkbox-box:before",
22+
".spectrum-Checkbox .spectrum-Checkbox-label",
1023
".spectrum-Checkbox .spectrum-Checkbox-partialCheckmark",
11-
".spectrum-Checkbox--emphasized .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
12-
".spectrum-Checkbox--emphasized .spectrum-Checkbox-input:focus-visible:checked + .spectrum-Checkbox-box:before",
1324
".spectrum-Checkbox--emphasized.is-indeterminate .spectrum-Checkbox-box:before",
1425
".spectrum-Checkbox--emphasized.is-indeterminate .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
15-
".spectrum-Checkbox--emphasized.is-indeterminate .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:before",
16-
".spectrum-Checkbox--emphasized.is-indeterminate:active .spectrum-Checkbox-box:before",
17-
".spectrum-Checkbox--emphasized.is-indeterminate:active .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
1826
".spectrum-Checkbox--emphasized.is-indeterminate:hover .spectrum-Checkbox-box:before",
1927
".spectrum-Checkbox--emphasized.is-indeterminate:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
20-
".spectrum-Checkbox--emphasized.is-invalid .spectrum-Checkbox-input:focus-visible:checked + .spectrum-Checkbox-box:before",
21-
".spectrum-Checkbox--emphasized.is-invalid.is-indeterminate:hover .spectrum-Checkbox-box:before",
22-
".spectrum-Checkbox--emphasized.is-invalid.is-indeterminate:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
23-
".spectrum-Checkbox--emphasized.is-invalid:active .spectrum-Checkbox-box:before",
24-
".spectrum-Checkbox--emphasized.is-invalid:active .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
25-
".spectrum-Checkbox--emphasized.is-invalid:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
26-
".spectrum-Checkbox--emphasized:active .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
27-
".spectrum-Checkbox--emphasized:focus-visible .spectrum-Checkbox-box:before",
28-
".spectrum-Checkbox--emphasized:focus-visible .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
29-
".spectrum-Checkbox--emphasized:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
30-
".spectrum-Checkbox--sizeL",
31-
".spectrum-Checkbox--sizeS",
32-
".spectrum-Checkbox--sizeXL",
33-
".spectrum-Checkbox-box",
34-
".spectrum-Checkbox-box:after",
35-
".spectrum-Checkbox-box:before",
36-
".spectrum-Checkbox-input",
37-
".spectrum-Checkbox-input:checked + .spectrum-Checkbox-box .spectrum-Checkbox-checkmark",
38-
".spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
39-
".spectrum-Checkbox-input:disabled",
28+
".spectrum-Checkbox--emphasized.is-indeterminate:not(.is-invalid) .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:before",
29+
".spectrum-Checkbox--emphasized:not(.is-invalid) .spectrum-Checkbox-input:focus-visible:checked + .spectrum-Checkbox-box:before",
4030
".spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box",
41-
".spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:after",
42-
".spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:before",
43-
".spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-label",
44-
".spectrum-Checkbox-input:focus-visible:checked + .spectrum-Checkbox-box:before",
45-
".spectrum-Checkbox-label",
46-
".spectrum-Checkbox-label:lang(ja)",
47-
".spectrum-Checkbox-label:lang(ko)",
48-
".spectrum-Checkbox-label:lang(zh)",
4931
".spectrum-Checkbox.is-indeterminate .spectrum-Checkbox-box .spectrum-Checkbox-checkmark",
5032
".spectrum-Checkbox.is-indeterminate .spectrum-Checkbox-box .spectrum-Checkbox-partialCheckmark",
5133
".spectrum-Checkbox.is-indeterminate .spectrum-Checkbox-box:before",
5234
".spectrum-Checkbox.is-indeterminate .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box .spectrum-Checkbox-checkmark",
5335
".spectrum-Checkbox.is-indeterminate .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box .spectrum-Checkbox-partialCheckmark",
5436
".spectrum-Checkbox.is-indeterminate .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
55-
".spectrum-Checkbox.is-indeterminate .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:before",
56-
".spectrum-Checkbox.is-indeterminate:hover .spectrum-Checkbox-box:before",
57-
".spectrum-Checkbox.is-indeterminate:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
58-
".spectrum-Checkbox.is-invalid .spectrum-Checkbox-box:before",
59-
".spectrum-Checkbox.is-invalid .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
60-
".spectrum-Checkbox.is-invalid .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:before",
37+
".spectrum-Checkbox.is-invalid",
6138
".spectrum-Checkbox.is-invalid.is-indeterminate .spectrum-Checkbox-box:before",
62-
".spectrum-Checkbox.is-invalid.is-indeterminate .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
39+
".spectrum-Checkbox.is-invalid.is-indeterminate .spectrum-Checkbox-input:checked:not(:disabled) + .spectrum-Checkbox-box:before",
6340
".spectrum-Checkbox.is-invalid.is-indeterminate .spectrum-Checkbox-input:focus-visible + .spectrum-Checkbox-box:before",
6441
".spectrum-Checkbox.is-invalid.is-indeterminate:hover .spectrum-Checkbox-box:before",
65-
".spectrum-Checkbox.is-invalid.is-indeterminate:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
66-
".spectrum-Checkbox.is-invalid.is-indeterminate:hover .spectrum-Checkbox-label",
67-
".spectrum-Checkbox.is-invalid:hover .spectrum-Checkbox-box:before",
68-
".spectrum-Checkbox.is-invalid:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
69-
".spectrum-Checkbox.is-readOnly .spectrum-Checkbox-input",
70-
".spectrum-Checkbox.is-readOnly .spectrum-Checkbox-input:checked:disabled + .spectrum-Checkbox-box:before",
71-
".spectrum-Checkbox.is-readOnly .spectrum-Checkbox-input:checked:disabled ~ .spectrum-Checkbox-label",
72-
".spectrum-Checkbox.is-readOnly .spectrum-Checkbox-input:disabled + .spectrum-Checkbox-box:before",
73-
".spectrum-Checkbox.is-readOnly .spectrum-Checkbox-input:disabled ~ .spectrum-Checkbox-label",
42+
".spectrum-Checkbox.is-invalid.is-indeterminate:hover .spectrum-Checkbox-input:checked:not(:disabled) + .spectrum-Checkbox-box:before",
43+
".spectrum-Checkbox.spectrum-Checkbox--emphasized",
44+
".spectrum-Checkbox.spectrum-Checkbox--sizeL",
45+
".spectrum-Checkbox.spectrum-Checkbox--sizeS",
46+
".spectrum-Checkbox.spectrum-Checkbox--sizeXL",
7447
".spectrum-Checkbox:active .spectrum-Checkbox-box:before",
7548
".spectrum-Checkbox:active .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
7649
".spectrum-Checkbox:active .spectrum-Checkbox-label",
7750
".spectrum-Checkbox:hover .spectrum-Checkbox-box:before",
7851
".spectrum-Checkbox:hover .spectrum-Checkbox-input:checked + .spectrum-Checkbox-box:before",
7952
".spectrum-Checkbox:hover .spectrum-Checkbox-label",
53+
".spectrum-Checkbox:lang(ja)",
54+
".spectrum-Checkbox:lang(ko)",
55+
".spectrum-Checkbox:lang(zh)",
56+
".spectrum-Checkbox:not(.is-readOnly) .spectrum-Checkbox-input:active:not(:disabled) + .spectrum-Checkbox-box",
8057
".spectrum-Checkbox:not(.is-readOnly):active .spectrum-Checkbox-input:not(:disabled) + .spectrum-Checkbox-box"
8158
],
8259
"modifiers": [
8360
"--mod-checkbox-animation-duration",
8461
"--mod-checkbox-border-width",
62+
"--mod-checkbox-bottom-to-text",
8563
"--mod-checkbox-checkmark-color",
8664
"--mod-checkbox-content-color-default",
8765
"--mod-checkbox-content-color-disabled",
@@ -94,38 +72,30 @@
9472
"--mod-checkbox-control-color-focus",
9573
"--mod-checkbox-control-color-hover",
9674
"--mod-checkbox-control-corner-radius",
97-
"--mod-checkbox-control-invalid-color-down",
9875
"--mod-checkbox-control-selected-color-default",
9976
"--mod-checkbox-control-selected-color-down",
100-
"--mod-checkbox-control-selected-color-focus",
10177
"--mod-checkbox-control-selected-color-hover",
10278
"--mod-checkbox-control-size",
103-
"--mod-checkbox-emphasized-color-default",
104-
"--mod-checkbox-emphasized-color-down",
105-
"--mod-checkbox-emphasized-color-focus",
106-
"--mod-checkbox-emphasized-color-hover",
10779
"--mod-checkbox-focus-indicator-color",
10880
"--mod-checkbox-focus-indicator-gap",
109-
"--mod-checkbox-focus-indicator-thinkness",
81+
"--mod-checkbox-focus-indicator-thickness",
11082
"--mod-checkbox-font-size",
11183
"--mod-checkbox-height",
112-
"--mod-checkbox-invalid-color-default",
113-
"--mod-checkbox-invalid-color-focus",
114-
"--mod-checkbox-invalid-color-hover",
11584
"--mod-checkbox-line-height",
11685
"--mod-checkbox-line-height-cjk",
86+
"--mod-checkbox-margin-block",
11787
"--mod-checkbox-selected-border-width",
118-
"--mod-checkbox-spacing",
11988
"--mod-checkbox-text-to-control",
12089
"--mod-checkbox-top-to-text",
12190
"--mod-focus-indicator-thickness"
12291
],
12392
"component": [
12493
"--spectrum-checkbox-animation-duration",
94+
"--spectrum-checkbox-background-color",
12595
"--spectrum-checkbox-border-width",
96+
"--spectrum-checkbox-bottom-to-text",
12697
"--spectrum-checkbox-checkmark-color",
12798
"--spectrum-checkbox-content-color-default",
128-
"--spectrum-checkbox-content-color-disabled",
12999
"--spectrum-checkbox-content-color-down",
130100
"--spectrum-checkbox-content-color-focus",
131101
"--spectrum-checkbox-content-color-hover",
@@ -137,7 +107,6 @@
137107
"--spectrum-checkbox-control-corner-radius",
138108
"--spectrum-checkbox-control-selected-color-default",
139109
"--spectrum-checkbox-control-selected-color-down",
140-
"--spectrum-checkbox-control-selected-color-focus",
141110
"--spectrum-checkbox-control-selected-color-hover",
142111
"--spectrum-checkbox-control-size",
143112
"--spectrum-checkbox-control-size-extra-large",
@@ -160,17 +129,28 @@
160129
"--spectrum-checkbox-line-height",
161130
"--spectrum-checkbox-line-height-cjk",
162131
"--spectrum-checkbox-selected-border-width",
163-
"--spectrum-checkbox-spacing",
132+
"--spectrum-checkbox-text-font-style",
133+
"--spectrum-checkbox-text-font-weight",
164134
"--spectrum-checkbox-text-to-control",
135+
"--spectrum-checkbox-top-to-control",
136+
"--spectrum-checkbox-top-to-control-extra-large",
137+
"--spectrum-checkbox-top-to-control-large",
138+
"--spectrum-checkbox-top-to-control-medium",
139+
"--spectrum-checkbox-top-to-control-small",
165140
"--spectrum-checkbox-top-to-text"
166141
],
167142
"global": [
168-
"--spectrum-accent-color-1000",
169-
"--spectrum-accent-color-1100",
170-
"--spectrum-accent-color-900",
143+
"--spectrum-accent-content-color-default",
144+
"--spectrum-accent-content-color-down",
145+
"--spectrum-accent-content-color-hover",
146+
"--spectrum-accent-content-color-key-focus",
171147
"--spectrum-animation-duration-100",
172148
"--spectrum-border-width-200",
173149
"--spectrum-cjk-line-height-100",
150+
"--spectrum-component-bottom-to-text-100",
151+
"--spectrum-component-bottom-to-text-200",
152+
"--spectrum-component-bottom-to-text-300",
153+
"--spectrum-component-bottom-to-text-75",
174154
"--spectrum-component-height-100",
175155
"--spectrum-component-height-200",
176156
"--spectrum-component-height-300",
@@ -183,7 +163,9 @@
183163
"--spectrum-component-top-to-text-75",
184164
"--spectrum-corner-radius-small-size-extra-large",
185165
"--spectrum-corner-radius-small-size-large",
166+
"--spectrum-corner-radius-small-size-medium",
186167
"--spectrum-corner-radius-small-size-small",
168+
"--spectrum-default-font-style",
187169
"--spectrum-disabled-content-color",
188170
"--spectrum-focus-indicator-color",
189171
"--spectrum-focus-indicator-gap",
@@ -192,21 +174,15 @@
192174
"--spectrum-font-size-200",
193175
"--spectrum-font-size-300",
194176
"--spectrum-font-size-75",
195-
"--spectrum-gray-50",
196-
"--spectrum-gray-700",
197-
"--spectrum-gray-800",
177+
"--spectrum-gray-25",
198178
"--spectrum-line-height-100",
199179
"--spectrum-negative-color-1000",
200-
"--spectrum-negative-color-1100",
201180
"--spectrum-negative-color-900",
202-
"--spectrum-neutral-background-color-selected-default",
203-
"--spectrum-neutral-background-color-selected-down",
204-
"--spectrum-neutral-background-color-selected-hover",
205-
"--spectrum-neutral-background-color-selected-key-focus",
206181
"--spectrum-neutral-content-color-default",
207182
"--spectrum-neutral-content-color-down",
208183
"--spectrum-neutral-content-color-hover",
209184
"--spectrum-neutral-content-color-key-focus",
185+
"--spectrum-regular-font-weight",
210186
"--spectrum-text-to-control-100",
211187
"--spectrum-text-to-control-200",
212188
"--spectrum-text-to-control-300",
@@ -217,7 +193,6 @@
217193
"--highcontrast-checkbox-background-color-default",
218194
"--highcontrast-checkbox-color-default",
219195
"--highcontrast-checkbox-color-focus",
220-
"--highcontrast-checkbox-color-hover",
221196
"--highcontrast-checkbox-content-color-default",
222197
"--highcontrast-checkbox-content-color-down",
223198
"--highcontrast-checkbox-content-color-focus",
@@ -226,7 +201,6 @@
226201
"--highcontrast-checkbox-focus-indicator-color",
227202
"--highcontrast-checkbox-highlight-color-default",
228203
"--highcontrast-checkbox-highlight-color-down",
229-
"--highcontrast-checkbox-highlight-color-focus",
230204
"--highcontrast-checkbox-highlight-color-hover"
231205
]
232206
}

0 commit comments

Comments
 (0)