-
Notifications
You must be signed in to change notification settings - Fork 11
Accessibilité: amélioration des boutons et des tags #664
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Je m'excuse un million de fois pour les sélecteurs très moches et les !important partout, je n'ai pas trouvé d'autre solution pour overrider les styles de base de Boostrap. Par ailleurs un sélecteur .focus-visible est ajouté à la compilation lorsque j'utilise :focus-visible, c'était peut-être utile en 2018 mais là c'est juste très très très pénible.
|
🔎 Download the Backstop report for this pull request (link valid for 90 days): |
|
Dans Molecules > Breadcrumb, section "tagged" Dans cette pull request, les tags sont légèrement remontés, ce qui fait que les textes dans le fil d'Ariane ne sont plus alignés. |
Je ne pense pas que c'était très dérangeant, mais c'était aussi corrigé en une ligne de CSS, c'est donc fait 🙂 |
|
@williambelle Pourrais-tu jeter un œil à cette PR et la merger si tout te parait en ordre ? Comme j'ai modifié des éléments 'atoms' de base, elle est bloquante pour d'autres travaux en cours. Merci beaucoup ! 😄 |
williambelle
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Est-ce que tu arrives à corriger les erreurs de l'Action 'Lint' ?
assets/components/atoms/tag/tag.scss
23:3 ✖ Expected "font-size" to come before "line-height" order/properties-order
89:5 ✖ Expected "width" to come before "line-height" order/properties-order
103:7 ✖ Expected "background" to come before "color" order/properties-order
134:9 ✖ Expected "background" to come before "color" order/properties-order
173:5 ✖ Expected "width" to come before "line-height" order/properties-order
Tu garantis la rétrocompatibilité du HTML avec ce nouveau CSS ?
|
@williambelle Les erreurs d'ordre de propriétés CSS sont corrigées. J'en ai profité pour refaire un tour des différents éléments du styleguide, et pour apporter quelques corrections / améliorations. Normalement il ne devrait pas y avoir de problème. |
Concernant les tags, je n'ai pas trouvé comment modifier les "tag input" qui sont générés en javascript. Le fichier tag-input.js ne contient pas la totalité du markup, et je ne trouve pas d'où provient le contenu de la balise . Ne sachant pas si cet élément est utilisé, j'ai préféré ne pas y passer trop de temps pour l'instant.