Skip to content
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

Amélioration de l'accessibilité - Issue #755 #926

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

hmartinez69
Copy link

@hmartinez69 hmartinez69 commented Jun 2, 2021

Relates #755

Résumé

Manque de rigueur sur la sémantique SEO des titres + button : réorganisation de l'ordre des titres et retrait de role=button

Détails

Aucun changement sur l'aspect visuel. Seulement une restructuration hiérarchique : un titre h1 par page, puis h2 puis h3

@hmartinez69 hmartinez69 mentioned this pull request Jun 2, 2021
9 tasks
@hmartinez69
Copy link
Author

Attention ici il s'agit bien d'un bouton (action locale, pas de navigation), le role est bien à sa place

Ok sorry. Je repush un commit now pour rétablir le role=button

@adrienpoly
Copy link
Collaborator

peux tu changer dans la description de la PR fixes par relates comme cette PR ne fix pas tous les problèmes de l'issue

après j'ai vu qu'à plusieurs endroit il y a une balise sémantique par exemple h2 mais une classe par exemple h4 je pense pas que cela soit un pb pour l'accessibilité c'est plus j'ai l'impression une problématique de notre style guide et de cohérence typographique

@carsso carsso marked this pull request as draft June 18, 2021 15:37
@hmartinez69 hmartinez69 marked this pull request as ready for review December 10, 2021 10:02
@hostolab hostolab deleted a comment from hmartinez69 Dec 13, 2022
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.

3 participants