Skip to content

Conversation

@alinekeller
Copy link
Collaborator

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.

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.
@alinekeller alinekeller requested review from williambelle and xentenza and removed request for williambelle May 30, 2023 14:40
@github-actions
Copy link

github-actions bot commented May 30, 2023

Unit Test Results

    1 files      1 suites   0s ⏱️
261 tests 253 ✔️ 0 💤 0  8 🔥
261 runs  245 ✔️ 0 💤 8  8 🔥

For more details on these errors, see this check.

Results for commit 790eec2.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented May 30, 2023

🔎 Download the Backstop report for this pull request (link valid for 90 days):

@xentenza
Copy link
Collaborator

xentenza commented Jun 7, 2023

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 sais pas à quel point c'est dérangeant).

@alinekeller
Copy link
Collaborator Author

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 sais pas à quel point c'est dérangeant).

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 🙂

@alinekeller
Copy link
Collaborator Author

@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 ! 😄

Copy link
Member

@williambelle williambelle left a 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 ?

@alinekeller
Copy link
Collaborator Author

@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.

@alinekeller alinekeller requested a review from williambelle July 4, 2023 14:01
@williambelle williambelle merged commit e3e34c2 into dev Jul 5, 2023
@williambelle williambelle deleted the styleguide/improve-accessibility branch July 5, 2023 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants