Skip to content

Commit 7094320

Browse files
fix(tag): expand clear button target area
- puts clear button spacing tokens into calc for clear button size in order to expand target area - removes older clear button calcs, modified calcs serve to cancel out unneeded spacing from item label
1 parent 7bef22a commit 7094320

File tree

2 files changed

+20
-7
lines changed

2 files changed

+20
-7
lines changed

components/tag/dist/metadata.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -245,6 +245,7 @@
245245
"--mod-clear-button-icon-color-hover",
246246
"--mod-clear-button-icon-color-key-focus",
247247
"--mod-clear-button-width",
248+
"--mod-icon-size",
248249
"--mod-thumbnail-opacity-disabled",
249250
"--mod-thumbnail-size"
250251
],

components/tag/index.css

Lines changed: 19 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -97,12 +97,25 @@
9797
--mod-avatar-inline-size: var(--mod-tag-avatar-size, var(--spectrum-tag-avatar-size));
9898
--mod-avatar-block-size: var(--mod-tag-avatar-size, var(--spectrum-tag-avatar-size));
9999
--mod-thumbnail-size: var(--mod-tag-thumbnail-size, var(--spectrum-tag-thumbnail-size));
100-
--mod-clear-button-width: var(--mod-tag-clear-button-size, var(--spectrum-tag-clear-button-size));
101-
--mod-clear-button-height: var(--mod-tag-clear-button-size, var(--spectrum-tag-clear-button-size));
102100
--mod-clear-button-background-color: transparent;
103101
--mod-clear-button-icon-color: currentColor;
104102
--mod-clear-button-icon-color-hover: currentColor;
105103
--mod-clear-button-icon-color-key-focus: currentColor;
104+
105+
--mod-icon-size: var(--mod-tag-clear-button-size, var(--spectrum-tag-clear-button-size));
106+
107+
/* clear icon inline spacing + icon size */
108+
--mod-clear-button-width: calc(
109+
var(--mod-tag-label-to-clear-icon, var(--spectrum-tag-label-to-clear-icon)) +
110+
var(--mod-tag-edge-to-clear-icon, var(--spectrum-tag-edge-to-clear-icon)) +
111+
var(--mod-tag-clear-button-size, var(--spectrum-tag-clear-button-size))
112+
);
113+
114+
/* clear icon block spacing + icon size */
115+
--mod-clear-button-height: calc(
116+
2 * var(--mod-tag-clear-button-spacing-block, var(--spectrum-tag-clear-button-spacing-block)) +
117+
var(--mod-tag-clear-button-size, var(--spectrum-tag-clear-button-size))
118+
);
106119
/* @passthrough end */
107120

108121
&:lang(ja),
@@ -228,12 +241,11 @@
228241
.spectrum-Tag-clearButton {
229242
box-sizing: border-box;
230243

231-
padding-block-start: calc(var(--mod-tag-clear-button-spacing-block, var(--spectrum-tag-clear-button-spacing-block)) - var(--mod-tag-border-width, var(--spectrum-tag-border-width)));
232-
padding-block-end: calc(var(--mod-tag-clear-button-spacing-block, var(--spectrum-tag-clear-button-spacing-block)) - var(--mod-tag-border-width, var(--spectrum-tag-border-width)));
244+
/* remove label inline-end spacing with negative margin */
245+
margin-inline-start: calc((var(--mod-tag-label-spacing-inline, var(--spectrum-tag-label-spacing-inline)) - var(--mod-tag-border-width, var(--spectrum-tag-border-width))) * -1);
233246

234-
/* move button negative calc to cancel label end margin, then add button start margin, add border width because it's being subtracted from the label spacing inline */
235-
margin-inline-start: calc(var(--mod-tag-label-to-clear-icon, var(--spectrum-tag-label-to-clear-icon)) - (var(--mod-tag-label-spacing-inline, var(--spectrum-tag-label-spacing-inline))) + var(--mod-tag-border-width, var(--spectrum-tag-border-width)));
236-
margin-inline-end: calc(var(--mod-tag-edge-to-clear-icon, var(--spectrum-tag-edge-to-clear-icon)) - var(--mod-tag-border-width, var(--spectrum-tag-border-width)));
247+
/* remove border width from clear button */
248+
margin-inline-end: calc(var(--mod-tag-border-width, var(--spectrum-tag-border-width)) * -1);
237249
}
238250

239251
.spectrum-Tag-itemLabel {

0 commit comments

Comments
 (0)