Skip to content

Commit 2261df6

Browse files
committed
refactor(clearbutton): remove unneeded CSS for overBackground and disabled
1 parent c39643a commit 2261df6

File tree

1 file changed

+10
-39
lines changed

1 file changed

+10
-39
lines changed

components/clearbutton/index.css

Lines changed: 10 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -35,13 +35,6 @@ governing permissions and limitations under the License.
3535
--spectrum-clear-button-width: var(--spectrum-component-height-300);
3636
}
3737

38-
&:disabled {
39-
--mod-clear-button-icon-color: var(--spectrum-clear-button-icon-color-disabled, var(--spectrum-disabled-content-color));
40-
--mod-clear-button-icon-color-hover: var(--spectrum-clear-button-icon-color-hover-disabled, var(--spectrum-disabled-content-color));
41-
--mod-clear-button-icon-color-down: var(--spectrum-clear-button-icon-color-down-disabled, var(--spectrum-disabled-content-color));
42-
--mod-clear-button-background-color: var(--spectrum-clear-button-background-color-disabled, var(--spectrum-disabled-background-color));
43-
}
44-
4538
&.spectrum-ClearButton--quiet {
4639
--mod-clear-button-background-color: var(--spectrum-clear-button-background-color-quiet, transparent);
4740
--mod-clear-button-background-color-hover: var(--spectrum-clear-button-background-color-hover-quiet, transparent);
@@ -58,7 +51,15 @@ governing permissions and limitations under the License.
5851
--mod-clear-button-background-color: var(--spectrum-clear-button-background-color-over-background, transparent);
5952
--mod-clear-button-background-color-hover: var(--spectrum-clear-button-background-color-hover-over-background, var(--spectrum-transparent-white-300));
6053
--mod-clear-button-background-color-down: var(--spectrum-clear-button-background-color-hover-over-background, var(--spectrum-transparent-white-400));
61-
--mod-clear-button-background-color-key-focus: var(--spectrum-clear-button-background-color-hover-over-background, var(--spectrum-transparent-white-300));
54+
--mod-clear-button-background-color-key-focus: var(--spectrum-clear-button-background-color-hover-over-background, var(--spectrum-transparent-white-300));
55+
}
56+
57+
&:disabled,
58+
&.is-disabled {
59+
--mod-clear-button-icon-color: var(--mod-clear-button-icon-color-disabled, var(--spectrum-disabled-content-color));
60+
--mod-clear-button-icon-color-hover: var(--spectrum-clear-button-icon-color-hover-disabled, var(--spectrum-disabled-content-color));
61+
--mod-clear-button-icon-color-down: var(--spectrum-clear-button-icon-color-down-disabled, var(--spectrum-disabled-content-color));
62+
--mod-clear-button-background-color: var(--mod-clear-button-background-color-disabled, transparent);
6263
}
6364
}
6465

@@ -68,7 +69,7 @@ governing permissions and limitations under the License.
6869
border-radius: 100%;
6970
cursor: pointer;
7071

71-
background-color: transparent;
72+
background-color: var(--mod-clear-button-background-color, transparent);
7273
margin: 0;
7374
padding: var(--mod-clear-button-padding, var(--spectrum-clear-button-padding));
7475

@@ -105,13 +106,6 @@ governing permissions and limitations under the License.
105106
background-color: var(--mod-clear-button-background-color-key-focus, var(--spectrum-clear-button-background-color-key-focus));
106107
}
107108
}
108-
109-
&:disabled,
110-
&.is-disabled {
111-
.spectrum-ClearButton-fill {
112-
background-color: var(--spectrum-clearbutton-fill-background-color-disabled);
113-
}
114-
}
115109
}
116110

117111
.spectrum-ClearButton-fill {
@@ -127,32 +121,9 @@ governing permissions and limitations under the License.
127121
}
128122

129123
.spectrum-ClearButton--overBackground {
130-
background-color: var(--spectrum-button-m-primary-outline-white-texticon-background-color);
131-
border-color: var(--spectrum-button-m-primary-outline-white-texticon-border-color);
132-
133124

134125
&:focus-visible {
135-
background-color: var(--spectrum-button-m-primary-outline-white-texticon-background-color-hover);
136-
border-color: var(--spectrum-button-m-primary-outline-white-texticon-border-color-hover);
137-
color: var(--spectrum-button-m-primary-outline-white-texticon-text-color-hover);
138-
box-shadow: none;
139126
outline: none;
140-
141-
&:after {
142-
box-shadow: 0 0 0 var(--spectrum-alias-focus-ring-size) var(--spectrum-button-m-primary-outline-white-texticon-border-color-key-focus);
143-
}
144-
}
145-
146-
&:disabled,
147-
&.is-disabled {
148-
background-color: var(--spectrum-button-m-primary-outline-white-texticon-background-color-disabled);
149-
border-color: var(--spectrum-button-m-primary-outline-white-texticon-border-color-disabled);
150-
color: var(--spectrum-button-m-primary-outline-white-texticon-text-color-disabled);
151-
}
152-
153-
&:disabled,
154-
&.is-disabled .spectrum-ClearButton-fill {
155-
background-color: var(--spectrum-alias-icon-color-overbackground-disabled);
156127
}
157128
}
158129

0 commit comments

Comments
 (0)