Improvement/Add style for focused element for accessibility #39
Conversation
scss/_button.scss
Outdated
| @@ -1,5 +1,5 @@ | |||
| button:focus { | |||
| outline: none !important; | |||
| *:focus, .focused, .checkbox-custom:focus::before, .flatpickr-day.inRange:focus { | |||
There was a problem hiding this comment.
Bootstrap a déjà une règle css de focus, il faut enlever cette règle qui la désactive.
There was a problem hiding this comment.
ha yes exact, mais je devrais pas laisser au moins celle ci pour qu'on ai notre couleur :
*:focus {
outline: 1px solid $blue-bright !important;
}
There was a problem hiding this comment.
Dans le fichier variables.scss:
$primary: $blue-bright;
$component-active-bg: theme-color("primary") !default;
$input-btn-focus-color: rgba($component-active-bg, .25) !default;
C'est donc déjà le blue-bright avec une opacité de .25
There was a problem hiding this comment.
ok parfait alors, j'ai retiré la rule du coup. Comme c'est prefix input, je pensais que ça ne le ferais pas sur les liens button ou autres select, mais ça semble good au final my bad.
There was a problem hiding this comment.
Top, et après par contre j'ai pas vérifié pour la checkbox-custom si c'était un input ou un btn, ni le inRange du flatpicker
There was a problem hiding this comment.
ce sont des label donc ça devrait être pris en compte à priori
No description provided.