Skip to content

Commit

Permalink
refactor(picker)!: remove focus-ring
Browse files Browse the repository at this point in the history
BREAKING CHANGE: Remove focus-ring pseudo class in favor of focus-visible
  • Loading branch information
mlogsdon18 authored and castastrophe committed Aug 16, 2023
1 parent 1bee468 commit e37b430
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions components/picker/index.css
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,6 @@ governing permissions and limitations under the License.
}
}
.spectrum-Picker--quiet {
&:focus-ring,
&:focus-visible,
&.is-focused {
forced-color-adjust: none;
Expand Down Expand Up @@ -279,7 +278,6 @@ governing permissions and limitations under the License.
}

&:focus-visible,
&:focus-ring,
&.is-focused {
outline: none;
background-color: var(--highcontrast-picker-background-color-default, var(--mod-picker-background-color-key-focus, var(--spectrum-picker-background-color-key-focus)));
Expand Down Expand Up @@ -345,7 +343,6 @@ governing permissions and limitations under the License.
border-color: var(--highcontrast-picker-border-color-error-hover-open, var(--mod-picker-border-color-error-hover-open, var(--spectrum-picker-border-color-error-hover-open)));
}

&:focus-ring,
&:focus-visible,
&.is-focused {
border-color: var(--highcontrast-picker-border-color-error-default, var(--mod-picker-border-color-error-key-focus, var(--spectrum-picker-border-color-error-key-focus)));
Expand Down Expand Up @@ -390,7 +387,6 @@ governing permissions and limitations under the License.

&:disabled,
&.is-disabled {
&:focus-ring,
&:focus-visible {
border-color: transparent;
}
Expand Down Expand Up @@ -497,7 +493,6 @@ governing permissions and limitations under the License.
background-color: transparent;
}

&:focus-ring,
&:focus-visible,
&.is-focused {
background-color: transparent;
Expand Down

0 comments on commit e37b430

Please sign in to comment.