Skip to content

Commit 43c551e

Browse files
authored
Outline active focus (#25145)
* Base the outline button :active color on the background, fixing a contrast issue * Only apply focus outline to active when it's focused
1 parent 5b8eaff commit 43c551e

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

scss/mixins/_buttons.scss

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,11 +79,14 @@
7979
&:not([disabled]):not(.disabled):active,
8080
&:not([disabled]):not(.disabled).active,
8181
.show > &.dropdown-toggle {
82-
color: color-yiq($color-hover);
82+
color: color-yiq($active-background);
8383
background-color: $active-background;
8484
border-color: $active-border;
85-
// Avoid using mixin so we can pass custom focus shadow properly
86-
box-shadow: 0 0 0 $btn-focus-width rgba($color, .5);
85+
86+
&:focus {
87+
// Avoid using mixin so we can pass custom focus shadow properly
88+
box-shadow: 0 0 0 $btn-focus-width rgba($color, .5);
89+
}
8790
}
8891
}
8992

0 commit comments

Comments
 (0)