Skip to content

Commit a0c8ee7

Browse files
committed
chore(swatch+swatchgroup): add focus ring
1 parent 0c0688b commit a0c8ee7

File tree

3 files changed

+15
-6
lines changed

3 files changed

+15
-6
lines changed

components/swatch/dist/metadata.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
".spectrum-Swatch.is-disabled",
3535
".spectrum-Swatch.is-disabled .spectrum-Swatch-disabledIcon",
3636
".spectrum-Swatch.is-image .spectrum-Swatch-fill:before",
37+
".spectrum-Swatch.is-keyboardFocused",
3738
".spectrum-Swatch.is-mixedValue",
3839
".spectrum-Swatch.is-mixedValue .spectrum-Swatch-fill",
3940
".spectrum-Swatch.is-mixedValue .spectrum-Swatch-icon",
@@ -46,6 +47,7 @@
4647
".spectrum-Swatch.is-selected:before",
4748
".spectrum-Swatch:after",
4849
".spectrum-Swatch:before",
50+
".spectrum-Swatch:focus-visible",
4951
".spectrum-Swatch:focus-visible:after",
5052
".spectrum-Swatch[disabled]",
5153
".spectrum-Swatch[disabled] .spectrum-Swatch-disabledIcon"
@@ -126,6 +128,7 @@
126128
"--spectrum-gray-25",
127129
"--spectrum-gray-900-rgb",
128130
"--spectrum-mixed-button-background",
131+
"--spectrum-mod-focus-indicator-color",
129132
"--spectrum-negative-visual-color",
130133
"--spectrum-neutral-content-color-default",
131134
"--spectrum-picked-color",

components/swatch/index.css

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,16 @@
3535
--spectrum-swatch-icon-color: var(--spectrum-neutral-content-color-default);
3636
--spectrum-swatch-slash-icon-color: var(--highcontrast-swatch-fill-foreground-color, var(--mod-swatch-slash-icon-color, var(--spectrum-negative-visual-color)));
3737
--spectrum-swatch-focus-indicator-color: var(--highcontrast-swatch-focus-indicator-color, var(--mod-swatch-focus-indicator-color, var(--spectrum-focus-indicator-color)));
38-
3938
--spectrum-mixed-button-background: var(--spectrum-gray-25);
40-
4139
--spectrum-add-button-background: var(--spectrum-gray-100);
4240
--spectrum-add-button-background-hover: var(--spectrum-gray-200);
4341
--spectrum-add-button-background-down: var(--spectrum-gray-200);
4442
--spectrum-add-button-background-keyboard-focus: var(--spectrum-gray-200);
43+
44+
/* Focus ring */
45+
--spectrum-swatch-focus-indicator-color: var(--spectrum-focus-indicator-color);
46+
--spectrum-swatch-focus-indicator-thickness: var(--spectrum-focus-indicator-thickness);
47+
--spectrum-swatch-focus-indicator-gap: var(--spectrum-focus-indicator-gap);
4548
}
4649

4750
.spectrum-Swatch--sizeXS {
@@ -123,6 +126,12 @@
123126
}
124127
}
125128

129+
&.is-keyboardFocused,
130+
&:focus-visible {
131+
outline: var(--mod-swatch-focus-indicator-thickness, var(--spectrum-swatch-focus-indicator-thickness)) solid var(--spectrum-mod-focus-indicator-color, var(--spectrum-swatch-focus-indicator-color));
132+
outline-offset: var(--mod-swatch-focus-indicator-gap, var(--spectrum-swatch-focus-indicator-gap));
133+
}
134+
126135
&.is-selected {
127136
background: var(--spectrum-swatch-inner-border-color-selected);
128137

components/swatch/stories/swatch.stories.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,10 +47,7 @@ export default {
4747
...isHovered,
4848
if: { arg: "isAddSwatch", truthy: true },
4949
},
50-
isKeyboardFocused: {
51-
...isKeyboardFocused,
52-
if: { arg: "isAddSwatch", truthy: true },
53-
},
50+
isKeyboardFocused,
5451
borderStyle: {
5552
name: "Border style",
5653
type: { name: "string" },

0 commit comments

Comments
 (0)