fix focus to use default focus to leave the browser's default focus sufficiently contrasted on all interactive elements#466
Merged
Conversation
…ufficiently contrasted on all interactive elements
Reviewer's GuideThis PR strips out manual focus styling across select components, SEO utilities, audio controls, skip links, and global focus rules—opting instead for the browser’s native focus ring to ensure consistent, high-contrast outlines. Class diagram for removed custom focus styles in SCSS utilities and componentsclassDiagram
class m-select-custom {
/* Focus styles removed: color, border-color, outline, box-shadow */
}
class seo-container {
/* No longer extends focus-seo-container */
}
class focus-seo-container {
/* Removed: outline and outline-offset for :focus-visible */
/* Removed: outline: none for *:focus */
}
class audio {
/* Removed: outline and outline-offset for :focus */
}
class skip-links {
/* Removed: outline-offset for :focus */
}
class global_focus {
/* outline-offset for :focus-visible changed from .5rem to .2rem */
}
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
francoistibo
approved these changes
Oct 6, 2025
MarieComet
approved these changes
Oct 6, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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-colorLe 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 que j'ai rencontrés sur absolument tous les projets audités.
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