-
Notifications
You must be signed in to change notification settings - Fork 2
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
[TECH] Autorise les lignes du tableaux à avoir une cellule de type heading scope "row" (PIX-16895) #838
Conversation
Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr838.review.pix.fr |
a93f1ed
to
a633108
Compare
0a0144c
to
a139f60
Compare
Pour moi tout est OK, mais petite question : à quoi ça sert ? Un heading ne suffit pas ? |
attributs scope accede . Ci dessous l'explication de l'attribut scope col / row pour les tableaux. ( c'est une évolution suite à ma formation speed training accessibilité que j'ai eu a Znk . ) |
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.
Tech review OK
@xav-car Merci pour les explications ! accessoirement, j'aurais jamais pensé aux tableaux à doubles entrées (qui nécessitent donc deux headers)
Même sans double entrée c'est aussi utile : cf Site WCAG21 - H63 |
a139f60
to
9e0aea8
Compare
## [54.14.1](v54.14.0...v54.14.1) (2025-03-07) ### 🏗️ Tech - [#838](#838) Autorise les lignes du tableaux à avoir une cellule de type heading scope "row" (PIX-16895)
🎉 This PR is included in version 54.14.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎄 Problème
Actuellement la tableau ne permet pas d'avoir sur une ligne une cellule de type heading
🎁 Proposition
Ajouter la possibilité d'avoir une cellule de type heading en la scopant avec
scope="row"
🌟 Remarques
RAS
🎅 Pour tester
Mettre le boolean isMainRow à true. et vérifier que nous avons bien un th pour chaque ligne avec le scope row