Skip to content

Commit 9b108f7

Browse files
fix(textfield): vertically center invalid icon (#3539)
1 parent 7bd5e4b commit 9b108f7

File tree

5 files changed

+45
-9
lines changed

5 files changed

+45
-9
lines changed

.changeset/healthy-windows-push.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"@spectrum-css/textfield": patch
3+
---
4+
5+
Updates invalid icon spacing to be vertically centered for S2.

components/textfield/dist/metadata.json

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,9 @@
8888
".spectrum-Textfield.is-valid .spectrum-Textfield-validationIcon",
8989
".spectrum-Textfield.spectrum-Textfield--quiet",
9090
".spectrum-Textfield.spectrum-Textfield--sideLabel .spectrum-Textfield-validationIcon",
91+
".spectrum-Textfield.spectrum-Textfield--sizeL",
92+
".spectrum-Textfield.spectrum-Textfield--sizeS",
93+
".spectrum-Textfield.spectrum-Textfield--sizeXL",
9194
".spectrum-Textfield:hover .spectrum-Textfield-input",
9295
".spectrum-Textfield:hover .spectrum-Textfield-input::placeholder"
9396
],
@@ -247,6 +250,10 @@
247250
"--spectrum-component-height-300",
248251
"--spectrum-component-height-75",
249252
"--spectrum-component-top-to-text-100",
253+
"--spectrum-component-top-to-workflow-icon-100",
254+
"--spectrum-component-top-to-workflow-icon-200",
255+
"--spectrum-component-top-to-workflow-icon-300",
256+
"--spectrum-component-top-to-workflow-icon-75",
250257
"--spectrum-corner-radius-100",
251258
"--spectrum-disabled-content-color",
252259
"--spectrum-field-edge-to-alert-icon-extra-large",
@@ -274,10 +281,6 @@
274281
"--spectrum-field-text-to-validation-icon-large",
275282
"--spectrum-field-text-to-validation-icon-medium",
276283
"--spectrum-field-text-to-validation-icon-small",
277-
"--spectrum-field-top-to-alert-icon-extra-large",
278-
"--spectrum-field-top-to-alert-icon-large",
279-
"--spectrum-field-top-to-alert-icon-medium",
280-
"--spectrum-field-top-to-alert-icon-small",
281284
"--spectrum-field-top-to-validation-icon-extra-large",
282285
"--spectrum-field-top-to-validation-icon-large",
283286
"--spectrum-field-top-to-validation-icon-medium",
@@ -338,7 +341,11 @@
338341
"--system-textfield-border-color-hover",
339342
"--system-textfield-border-color-keyboard-focus",
340343
"--system-textfield-border-width",
341-
"--system-textfield-quiet-border-color-disabled"
344+
"--system-textfield-icon-spacing-block-invalid",
345+
"--system-textfield-quiet-border-color-disabled",
346+
"--system-textfield-size-l-icon-spacing-block-invalid",
347+
"--system-textfield-size-s-icon-spacing-block-invalid",
348+
"--system-textfield-size-xl-icon-spacing-block-invalid"
342349
],
343350
"passthroughs": [],
344351
"high-contrast": [

components/textfield/index.css

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,6 @@
9999
--spectrum-textfield-icon-size-valid: var(--spectrum-checkmark-icon-size-100);
100100
--spectrum-textfield-icon-spacing-inline-end-invalid: var(--spectrum-field-edge-to-alert-icon-medium);
101101
--spectrum-textfield-icon-spacing-inline-end-valid: var(--spectrum-field-edge-to-validation-icon-medium);
102-
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-medium);
103102
--spectrum-textfield-icon-spacing-block-valid: var(--spectrum-field-top-to-validation-icon-medium);
104103
--spectrum-textfield-icon-spacing-inline-start-invalid: var(--spectrum-field-text-to-alert-icon-medium);
105104
--spectrum-textfield-icon-spacing-inline-start-valid: var(--spectrum-field-text-to-validation-icon-medium);
@@ -123,7 +122,6 @@
123122
--spectrum-textfield-icon-size-valid: var(--spectrum-checkmark-icon-size-75);
124123
--spectrum-textfield-icon-spacing-inline-end-invalid: var(--spectrum-field-edge-to-alert-icon-small);
125124
--spectrum-textfield-icon-spacing-inline-end-valid: var(--spectrum-field-edge-to-validation-icon-small);
126-
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-small);
127125
--spectrum-textfield-icon-spacing-block-valid: var(--spectrum-field-top-to-validation-icon-small);
128126
--spectrum-textfield-icon-spacing-inline-start-invalid: var(--spectrum-field-text-to-alert-icon-small);
129127
--spectrum-textfield-icon-spacing-inline-start-valid: var(--spectrum-field-text-to-validation-icon-small);
@@ -146,7 +144,6 @@
146144
--spectrum-textfield-icon-size-valid: var(--spectrum-checkmark-icon-size-200);
147145
--spectrum-textfield-icon-spacing-inline-end-invalid: var(--spectrum-field-edge-to-alert-icon-large);
148146
--spectrum-textfield-icon-spacing-inline-end-valid: var(--spectrum-field-edge-to-validation-icon-large);
149-
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-large);
150147
--spectrum-textfield-icon-spacing-block-valid: var(--spectrum-field-top-to-validation-icon-large);
151148
--spectrum-textfield-icon-spacing-inline-start-invalid: var(--spectrum-field-text-to-alert-icon-large);
152149
--spectrum-textfield-icon-spacing-inline-start-valid: var(--spectrum-field-text-to-validation-icon-large);
@@ -169,7 +166,6 @@
169166
--spectrum-textfield-icon-size-valid: var(--spectrum-checkmark-icon-size-300);
170167
--spectrum-textfield-icon-spacing-inline-end-invalid: var(--spectrum-field-edge-to-alert-icon-extra-large);
171168
--spectrum-textfield-icon-spacing-inline-end-valid: var(--spectrum-field-edge-to-validation-icon-extra-large);
172-
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-extra-large);
173169
--spectrum-textfield-icon-spacing-block-valid: var(--spectrum-field-top-to-validation-icon-extra-large);
174170
--spectrum-textfield-icon-spacing-inline-start-invalid: var(--spectrum-field-text-to-alert-icon-extra-large);
175171
--spectrum-textfield-icon-spacing-inline-start-valid: var(--spectrum-field-text-to-validation-icon-extra-large);

components/textfield/themes/spectrum-two.css

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,20 @@
2525

2626
--spectrum-textfield-border-width: var(--spectrum-border-width-100);
2727

28+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-component-top-to-workflow-icon-100);
29+
30+
&.spectrum-Textfield--sizeS {
31+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-component-top-to-workflow-icon-75);
32+
}
33+
34+
&.spectrum-Textfield--sizeL {
35+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-component-top-to-workflow-icon-200);
36+
}
37+
38+
&.spectrum-Textfield--sizeXL {
39+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-component-top-to-workflow-icon-300);
40+
}
41+
2842
&.spectrum-Textfield--quiet {
2943
--spectrum-textfield-border-color-disabled: var(--spectrum-gray-300);
3044
}

components/textfield/themes/spectrum.css

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,20 @@
2222

2323
--spectrum-textfield-border-color-disabled: var(--spectrum-gray-200);
2424

25+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-medium);
26+
27+
&.spectrum-Textfield--sizeS {
28+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-small);
29+
}
30+
31+
&.spectrum-Textfield--sizeL {
32+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-large);
33+
}
34+
35+
&.spectrum-Textfield--sizeXL {
36+
--spectrum-textfield-icon-spacing-block-invalid: var(--spectrum-field-top-to-alert-icon-extra-large);
37+
}
38+
2539
&.spectrum-Textfield--quiet {
2640
--spectrum-textfield-border-color-disabled: var(--spectrum-gray-300);
2741
}

0 commit comments

Comments
 (0)