Commit 484b4f11 authored by Lyza Danger Gardner's avatar Lyza Danger Gardner Committed by Lyza Gardner

Apply hover styling consistently to focused buttons

The recently-added `button-hover` mixin, used by the base `button`
mixin and thus inherited by other button mixins, applied hover styling
to focused buttons, but other button mixins that adjust hover styling
were not applying hover styling to focused buttons.

Update mixins to consistently apply hovered styling to focused buttons

Fixes #2610
parent bf9a06fc
......@@ -69,7 +69,8 @@
&[aria-pressed='true'] {
color: var.$color-brand;
&:hover:not([disabled]) {
&:hover:not([disabled]),
&:focus:not([disabled]) {
color: var.$color-brand;
}
}
......@@ -84,7 +85,8 @@
font-weight: 700;
background-color: var.$grey-1;
&:hover:not([disabled]) {
&:hover:not([disabled]),
&:focus:not([disabled]) {
background-color: var.$grey-2;
}
......@@ -103,7 +105,8 @@
color: var.$color-text--inverted;
background-color: var.$grey-mid;
&:hover:not([disabled]) {
&:hover:not([disabled]),
&:focus:not([disabled]) {
color: var.$color-text--inverted;
background-color: var.$grey-6;
}
......@@ -137,7 +140,8 @@
border-radius: 0; // Turn off border-radius to align with <input> edges
border-left: 0; // Avoid double border with the <input>
&:hover:not([disabled]) {
&:hover:not([disabled]),
&:focus:not([disabled]) {
background-color: var.$grey-2;
}
}
......@@ -149,7 +153,8 @@
@include button;
color: var.$grey-mid;
padding: 0;
&:hover:not([disabled]) {
&:hover:not([disabled]),
&:focus:not([disabled]) {
color: var.$color-link-hover;
text-decoration: underline;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment