Skip to content

Commit 9be7d0f

Browse files
fix(TextInput): fixes hover, focus issues with visibility icon (#7680)
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
1 parent b3b5a02 commit 9be7d0f

File tree

1 file changed

+22
-4
lines changed

1 file changed

+22
-4
lines changed

packages/components/src/components/text-input/_text-input.scss

Lines changed: 22 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -100,17 +100,25 @@
100100

101101
.#{$prefix}--text-input--password__visibility,
102102
// TODO: remove selector above
103-
.#{$prefix}--text-input--password__visibility__toggle.#{$prefix}--tooltip__trigger {
103+
.#{$prefix}--btn.#{$prefix}--text-input--password__visibility__toggle.#{$prefix}--tooltip__trigger {
104+
@include focus-outline('reset');
105+
104106
position: absolute;
105-
right: $carbon--spacing-05;
106-
width: rem(16px);
107-
height: rem(16px);
107+
right: 0;
108+
display: flex;
109+
align-items: center;
110+
justify-content: center;
111+
width: rem(40px);
112+
height: rem(40px);
113+
min-height: auto;
108114
padding: 0;
109115
background: none;
110116
border: 0;
111117
cursor: pointer;
118+
transition: outline $duration--fast-01 motion(standard, productive);
112119

113120
svg {
121+
transition: fill $duration--fast-01 motion(standard, productive);
114122
fill: $icon-02;
115123

116124
// Windows, Firefox HCM Fix
@@ -122,6 +130,16 @@
122130
}
123131
}
124132

133+
.#{$prefix}--btn.#{$prefix}--text-input--password__visibility__toggle.#{$prefix}--tooltip__trigger:focus {
134+
@include focus-outline('outline');
135+
}
136+
137+
.#{$prefix}--btn.#{$prefix}--text-input--password__visibility__toggle.#{$prefix}--tooltip__trigger:hover,
138+
.#{$prefix}--btn.#{$prefix}--text-input--password__visibility__toggle.#{$prefix}--tooltip__trigger:focus
139+
svg {
140+
fill: $icon-01;
141+
}
142+
125143
.#{$prefix}--text-input--invalid {
126144
padding-right: $carbon--spacing-08;
127145
}

0 commit comments

Comments
 (0)