search

cap-collectif / ui

Design system of @cap-collectif with react • styled-components • styled-system
https://ds.cap-collectif.com/
MIT License
16 stars 0 forks source link

feat(Accessibility): auto assign id and correct html tag on modal header #435

Closed aguilaj closed 4 months ago

aguilaj commented 4 months ago

:tophat: What? Why?

Le bouton de fermeture doit référencer le titre de la modale. En terme de conception, s'il y a une étiquette, on veut que ce soit elle le titre Sinon on veut que ce soit le titre normal. Donc je check les children du header et j'assigne au besoin. Un peu bas niveau mais ça rend la modale utilisable et accessible sans avoir à le gérer côté platform

:pushpin: Related Issues

seblepers commented 4 months ago

En titre de PR, tu pourrais écrire comme ça ton "conventional commit" :

feat(Accessibility): auto assign id and correct html tag on modal header

myracodes commented 4 months ago

En titre de PR, tu pourrais écrire comme ça ton "conventional commit" :

feat(Accessibility): auto assign id and correct html tag on modal header

Je note de prendre le réflexe d'ajouter le scope dans ce format dans mes titres de commit aussi !