Skip to content

Commit a8bb0b8

Browse files
* fix(actionbutton): s2 foundations fast follows (#3540)
* fix(actionbutton): define action button border radius for s1/s2 * fix(actionbutton): defines disabled background colors - defines --spectrum-actionbutton-background-color-disabled, --spectrum-actionbutton-background-color-selected-disabled custom properties to use disabled-background-color and disabled-static-*- background-color is those states - updates metadata.json - S1 actionbutton disabled background color should not have been altered - preserve express disabled `gray-200` background color, and updates background in selected-disabled dark mode (to use disabled-background- color) * fix(actionbutton): transparent background for S1 disabled state * chore(actionbutton): add changeset
1 parent 03dce55 commit a8bb0b8

File tree

6 files changed

+77
-12
lines changed

6 files changed

+77
-12
lines changed

.changeset/quick-hairs-walk.md

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
"@spectrum-css/actionbutton": patch
3+
---
4+
5+
S2 Foundations fixes
6+
7+
- Adjusts corner rounding for S2 foundations to follow S2 action button corner rounding, particularly to help give additional space to the help/corner triangle icon:
8+
- corner-radius-medium-size-extra-small
9+
- corner-radius-medium-size-small
10+
- corner-radius-medium-size-medium
11+
- corner-radius-medium-size-large
12+
- corner-radius-medium-size-extra-large
13+
- Corrects the `disabled-background-color` and `disabled-static-*-background-color` tokens for the disabled and selected+disabled states

components/actionbutton/dist/metadata.json

Lines changed: 23 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,11 @@
2020
".spectrum-ActionButton.is-selected .spectrum-ActionButton-label",
2121
".spectrum-ActionButton.spectrum-ActionButton--emphasized:not(.spectrum-ActionButton--staticBlack, .spectrum-ActionButton--staticWhite)",
2222
".spectrum-ActionButton.spectrum-ActionButton--quiet",
23+
".spectrum-ActionButton.spectrum-ActionButton--sizeL",
24+
".spectrum-ActionButton.spectrum-ActionButton--sizeM",
25+
".spectrum-ActionButton.spectrum-ActionButton--sizeS",
26+
".spectrum-ActionButton.spectrum-ActionButton--sizeXL",
27+
".spectrum-ActionButton.spectrum-ActionButton--sizeXS",
2328
".spectrum-ActionButton.spectrum-ActionButton--staticBlack",
2429
".spectrum-ActionButton.spectrum-ActionButton--staticBlack.spectrum-ActionButton--quiet",
2530
".spectrum-ActionButton.spectrum-ActionButton--staticWhite",
@@ -165,11 +170,18 @@
165170
"--spectrum-component-height-300",
166171
"--spectrum-component-height-50",
167172
"--spectrum-component-height-75",
168-
"--spectrum-corner-radius-100",
173+
"--spectrum-corner-radius-medium-size-extra-large",
174+
"--spectrum-corner-radius-medium-size-extra-small",
175+
"--spectrum-corner-radius-medium-size-large",
176+
"--spectrum-corner-radius-medium-size-medium",
177+
"--spectrum-corner-radius-medium-size-small",
178+
"--spectrum-disabled-background-color",
169179
"--spectrum-disabled-border-color",
170180
"--spectrum-disabled-content-color",
181+
"--spectrum-disabled-static-black-background-color",
171182
"--spectrum-disabled-static-black-border-color",
172183
"--spectrum-disabled-static-black-content-color",
184+
"--spectrum-disabled-static-white-background-color",
173185
"--spectrum-disabled-static-white-border-color",
174186
"--spectrum-disabled-static-white-content-color",
175187
"--spectrum-focus-indicator-color",
@@ -204,15 +216,12 @@
204216
"--spectrum-text-to-visual-300",
205217
"--spectrum-text-to-visual-50",
206218
"--spectrum-text-to-visual-75",
207-
"--spectrum-transparent-black-100",
208-
"--spectrum-transparent-black-200",
209219
"--spectrum-transparent-black-400",
210220
"--spectrum-transparent-black-500",
211221
"--spectrum-transparent-black-600",
212222
"--spectrum-transparent-black-700",
213223
"--spectrum-transparent-black-800",
214224
"--spectrum-transparent-black-900",
215-
"--spectrum-transparent-white-200",
216225
"--spectrum-transparent-white-400",
217226
"--spectrum-transparent-white-500",
218227
"--spectrum-transparent-white-600",
@@ -228,11 +237,11 @@
228237
],
229238
"system-theme": [
230239
"--system-action-button-background-color-default",
240+
"--system-action-button-background-color-disabled",
231241
"--system-action-button-background-color-down",
232242
"--system-action-button-background-color-focus",
233243
"--system-action-button-background-color-hover",
234244
"--system-action-button-background-color-selected",
235-
"--system-action-button-background-color-selected-disabled",
236245
"--system-action-button-background-color-selected-down",
237246
"--system-action-button-background-color-selected-focus",
238247
"--system-action-button-background-color-selected-hover",
@@ -243,9 +252,16 @@
243252
"--system-action-button-border-color-hover",
244253
"--system-action-button-content-color-selected",
245254
"--system-action-button-quiet-background-color-default",
255+
"--system-action-button-quiet-background-color-disabled",
246256
"--system-action-button-quiet-background-color-down",
247257
"--system-action-button-quiet-background-color-focus",
248258
"--system-action-button-quiet-background-color-hover",
259+
"--system-action-button-quiet-background-color-selected-disabled",
260+
"--system-action-button-size-l-border-radius",
261+
"--system-action-button-size-m-border-radius",
262+
"--system-action-button-size-s-border-radius",
263+
"--system-action-button-size-xl-border-radius",
264+
"--system-action-button-size-xs-border-radius",
249265
"--system-action-button-static-black-background-color-default",
250266
"--system-action-button-static-black-background-color-disabled",
251267
"--system-action-button-static-black-background-color-down",
@@ -257,6 +273,7 @@
257273
"--system-action-button-static-black-border-color-focus",
258274
"--system-action-button-static-black-border-color-hover",
259275
"--system-action-button-static-black-quiet-background-color-default",
276+
"--system-action-button-static-black-quiet-background-color-disabled",
260277
"--system-action-button-static-black-quiet-background-color-down",
261278
"--system-action-button-static-black-quiet-background-color-focus",
262279
"--system-action-button-static-black-quiet-background-color-hover",
@@ -271,6 +288,7 @@
271288
"--system-action-button-static-white-border-color-focus",
272289
"--system-action-button-static-white-border-color-hover",
273290
"--system-action-button-static-white-quiet-background-color-default",
291+
"--system-action-button-static-white-quiet-background-color-disabled",
274292
"--system-action-button-static-white-quiet-background-color-down",
275293
"--system-action-button-static-white-quiet-background-color-focus",
276294
"--system-action-button-static-white-quiet-background-color-hover"

components/actionbutton/index.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
--spectrum-actionbutton-border-color: var(--highcontrast-actionbutton-border-color, var(--mod-actionbutton-border-color-default, var(--spectrum-actionbutton-border-color-default)));
5454
--spectrum-actionbutton-content-color: var(--highcontrast-actionbutton-content-color, var(--mod-actionbutton-content-color-default, var(--spectrum-neutral-content-color-default)));
5555

56-
--spectrum-actionbutton-border-radius: var(--mod-actionbutton-border-radius, var(--spectrum-corner-radius-100));
56+
--spectrum-actionbutton-border-radius: var(--mod-actionbutton-border-radius, var(--spectrum-actionbutton-border-radius));
5757
--spectrum-actionbutton-border-width: var(--mod-actionbutton-border-width, var(--spectrum-border-width-100));
5858

5959
--spectrum-actionbutton-focus-indicator-gap: var(--mod-actionbutton-focus-indicator-gap, var(--spectrum-focus-indicator-gap));

components/actionbutton/themes/express.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
--spectrum-actionbutton-background-color-hover: var(--spectrum-gray-300);
2222
--spectrum-actionbutton-background-color-down: var(--spectrum-gray-400);
2323
--spectrum-actionbutton-background-color-focus: var(--spectrum-gray-300);
24+
--spectrum-actionbutton-background-color-disabled: var(--spectrum-gray-200);
2425

2526
--spectrum-actionbutton-border-color-default: transparent;
2627
--spectrum-actionbutton-border-color-hover: transparent;

components/actionbutton/themes/spectrum-two.css

Lines changed: 29 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
--spectrum-actionbutton-background-color-hover: var(--spectrum-gray-100);
1818
--spectrum-actionbutton-background-color-down: var(--spectrum-gray-200);
1919
--spectrum-actionbutton-background-color-focus: var(--spectrum-gray-100);
20-
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-transparent-black-100);
20+
--spectrum-actionbutton-background-color-disabled: var(--spectrum-disabled-background-color);
2121

2222
--spectrum-actionbutton-background-color-selected: var(--spectrum-neutral-background-color-selected-default);
2323
--spectrum-actionbutton-background-color-selected-hover: var(--spectrum-neutral-background-color-selected-hover);
@@ -32,11 +32,33 @@
3232

3333
--spectrum-actionbutton-content-color-selected: var(--spectrum-gray-50);
3434

35+
&.spectrum-ActionButton--sizeM {
36+
--spectrum-actionbutton-border-radius: var(--spectrum-corner-radius-medium-size-medium);
37+
}
38+
39+
&.spectrum-ActionButton--sizeXS {
40+
--spectrum-actionbutton-border-radius: var(--spectrum-corner-radius-medium-size-extra-small);
41+
}
42+
43+
&.spectrum-ActionButton--sizeS {
44+
--spectrum-actionbutton-border-radius: var(--spectrum-corner-radius-medium-size-small);
45+
}
46+
47+
&.spectrum-ActionButton--sizeL {
48+
--spectrum-actionbutton-border-radius: var(--spectrum-corner-radius-medium-size-large);
49+
}
50+
51+
&.spectrum-ActionButton--sizeXL {
52+
--spectrum-actionbutton-border-radius: var(--spectrum-corner-radius-medium-size-extra-large);
53+
}
54+
3555
&.spectrum-ActionButton--quiet {
3656
--spectrum-actionbutton-background-color-default: transparent;
3757
--spectrum-actionbutton-background-color-hover: var(--spectrum-gray-100);
3858
--spectrum-actionbutton-background-color-down: var(--spectrum-gray-200);
3959
--spectrum-actionbutton-background-color-focus: var(--spectrum-gray-100);
60+
--spectrum-actionbutton-background-color-disabled: transparent;
61+
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-disabled-background-color);
4062
}
4163

4264
&.spectrum-ActionButton--staticBlack {
@@ -45,15 +67,16 @@
4567
--spectrum-actionbutton-border-color-down: var(--spectrum-transparent-black-700);
4668
--spectrum-actionbutton-border-color-focus: var(--spectrum-transparent-black-600);
4769

48-
--spectrum-actionbutton-background-color-disabled: transparent;
49-
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-transparent-black-200);
70+
--spectrum-actionbutton-background-color-disabled: var(--spectrum-disabled-static-black-background-color);
71+
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-disabled-static-black-background-color);
5072

5173
&,
5274
&.spectrum-ActionButton--quiet {
5375
--spectrum-actionbutton-background-color-default: transparent;
5476
--spectrum-actionbutton-background-color-hover: var(--spectrum-transparent-black-400);
5577
--spectrum-actionbutton-background-color-down: var(--spectrum-transparent-black-500);
5678
--spectrum-actionbutton-background-color-focus: var(--spectrum-transparent-black-400);
79+
--spectrum-actionbutton-background-color-disabled: transparent;
5780
}
5881
}
5982

@@ -63,15 +86,16 @@
6386
--spectrum-actionbutton-border-color-down: var(--spectrum-transparent-white-700);
6487
--spectrum-actionbutton-border-color-focus: var(--spectrum-transparent-white-600);
6588

66-
--spectrum-actionbutton-background-color-disabled: transparent;
67-
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-transparent-white-200);
89+
--spectrum-actionbutton-background-color-disabled: var(--spectrum-disabled-static-white-background-color);
90+
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-disabled-static-white-background-color);
6891

6992
&,
7093
&.spectrum-ActionButton--quiet {
7194
--spectrum-actionbutton-background-color-default: transparent;
7295
--spectrum-actionbutton-background-color-hover: var(--spectrum-transparent-white-400);
7396
--spectrum-actionbutton-background-color-down: var(--spectrum-transparent-white-500);
7497
--spectrum-actionbutton-background-color-focus: var(--spectrum-transparent-white-400);
98+
--spectrum-actionbutton-background-color-disabled: transparent;
7599
}
76100
}
77101
}

components/actionbutton/themes/spectrum.css

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,19 @@
2121
--spectrum-actionbutton-background-color-hover: var(--spectrum-gray-200);
2222
--spectrum-actionbutton-background-color-down: var(--spectrum-gray-300);
2323
--spectrum-actionbutton-background-color-focus: var(--spectrum-gray-200);
24-
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-transparent-black-200);
24+
--spectrum-actionbutton-background-color-disabled: transparent;
25+
--spectrum-actionbutton-background-color-selected-disabled: var(--spectrum-disabled-background-color);
2526

2627
--spectrum-actionbutton-content-color-selected: var(--spectrum-gray-75);
2728

29+
&.spectrum-ActionButton--sizeXS,
30+
&.spectrum-ActionButton--sizeS,
31+
&.spectrum-ActionButton--sizeM,
32+
&.spectrum-ActionButton--sizeL,
33+
&.spectrum-ActionButton--sizeXL {
34+
--spectrum-actionbutton-border-radius: var(--spectrum-corner-radius-100);
35+
}
36+
2837
&.spectrum-ActionButton--quiet {
2938
--spectrum-actionbutton-background-color-hover: var(--spectrum-gray-200);
3039
--spectrum-actionbutton-background-color-down: var(--spectrum-gray-300);

0 commit comments

Comments
 (0)