fix focus to use default focus to leave the browser's default focus sufficiently contrasted on all interactive elements #466
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Suite à de nombreux retours sur des audits a11y, il vaut mieux garder le focus natif navigateur qui automatiquement va garder son anneau de focus suffisamment contrasté par rapport à la couleur de fond (il faut un contraste de minimum 3:1). Webkit utilise par exemple
webkit-focus-ring-color
Le fait d'avoir un currentColor pose souvent des soucis, par exemple dans le cas d'un élément blanc sur fond de couleur, donc le focus n'est pas visible, et il faut penser à apporter des surcharges au cas par cas. et en plus s'assurer que le contraste soit suffisant.
C'est typiquement les soucis qu'on a sur plusieurs projets.
On garde également le focus uniquement sur le lien présent dans un card et non la card complète, ce qui apporte plus de contexte et aussi parfois on a des taxo qui sont cliquables dans les cards et donc le focus n'est pas visible.
On se contente juste de jouer avec l'offset et la taille de l'outline au besoin