Deuxfleurs/site
20
2
Fork 6
Code Issues 4 Pull requests 7 Releases Wiki Activity

Acessibilité web: Améliorations sémantique #42

Merged
tixie merged 4 commits from tixie/deuxfleurs-website:a11y/semantic into preprod 2024-06-08 19:08:57 +00:00
Conversation 2 Commits 4 Files changed 4 +67 -24
tixie commented 2024-06-07 18:10:36 +00:00
Owner
Copy link

Bonjour,
Suite au super travail mené par @adrian j'ai fait une pull request avec un premier lot de modifications, celles si ne changeant que la sémantique du HTML, et donc normalement ne doit pas changer de comportement visuel.

Cette PR contient :

  1. la suppression de l'attribut aria-orientation="vertical" dans la navigation : Inutile et faux dans le cas où le site est visionné sur un navigateur à petite largeur (smartphone par exemple)
  2. la suppression des role="none" sur les élement de liste: il est important de garder une sémantique dans les items de liste. Je ne vois pas pourquoi ces attributs ont été ajoutés.
  3. l'illustration de l'ordinateur portable s'est vu rajouté le role img et un aria-label qui manquait comme ses autres sœurs illustrations
  4. Une balise div avait oubliée d'être fermée après le calendrier
  5. Et le plus important pour la fin: Les soulignements de titres (une suite de 70 caractères underscore et guillemets) étaient lus par les lecteurs d'écran, pas vraiment quelque chose de souhaitable (oui le contenu de la propriété content en CSS est lu par les lecteurs d'écrans)
  6. L'ajout du rapport écrit par Adrian, qui pourra peut-être trouver sa place quelque part mais en attendant sera ainsi disponible sur la forge git.

Cette PR s'occupe donc de résoudre les issues #29, #28, #27

J'ai déployé l'ensemble des modifications sur cette démo https://a11y-semantic.deuxfleurs.guerilla.studio

Je vois les modifications visuelles: aka le travail de play/pause des animations et du respect de la media feature prefers-reduced-motion pour une seconde PR, comme elle implique des changements visuels, donc impliquera potentiellement plus d'aller-retour.

Mais en attendant que vous pensez de ces modifs @esther @adrien ? Est-ce que c'est quelque chose qui pourrait être poussé sur la version en ligne du site ?

Bonjour, Suite au [super travail](https://git.deuxfleurs.fr/adrian/site/src/branch/main) mené par @adrian j'ai fait une pull request avec un premier lot de modifications, celles si ne changeant que la sémantique du HTML, et donc normalement ne doit pas changer de comportement visuel. Cette PR contient : 1. la suppression de l'attribut `aria-orientation="vertical"` dans la navigation : Inutile et faux dans le cas où le site est visionné sur un navigateur à petite largeur (smartphone par exemple) 2. la suppression des `role="none"` sur les élement de liste: il est important de garder une sémantique dans les items de liste. Je ne vois pas pourquoi ces attributs ont été ajoutés. 3. l'illustration de l'ordinateur portable s'est vu rajouté le role img et un aria-label qui manquait comme ses autres sœurs illustrations 4. Une balise div avait oubliée d'être fermée après le calendrier 5. Et le plus important pour la fin: Les soulignements de titres (une suite de 70 caractères underscore et guillemets) étaient lus par les lecteurs d'écran, pas vraiment quelque chose de souhaitable (oui le contenu de la propriété `content` en CSS est lu par les lecteurs d'écrans) 6. L'ajout du rapport écrit par Adrian, qui pourra peut-être trouver sa place quelque part mais en attendant sera ainsi disponible sur la forge git. Cette PR s'occupe donc de résoudre les issues #29, #28, #27 J'ai déployé l'ensemble des modifications sur cette démo https://a11y-semantic.deuxfleurs.guerilla.studio Je vois les modifications visuelles: aka le travail de play/pause des animations et du respect de la media feature `prefers-reduced-motion` pour une seconde PR, comme elle implique des changements visuels, donc impliquera potentiellement plus d'aller-retour. Mais en attendant que vous pensez de ces modifs @esther @adrien ? Est-ce que c'est quelque chose qui pourrait être poussé sur la version en ligne du site ?
tixie self-assigned this 2024-06-07 18:10:36 +00:00
adrien was assigned by tixie 2024-06-07 18:10:36 +00:00
tixie requested review from adrien 2024-06-07 18:10:51 +00:00
tixie requested review from esther 2024-06-07 18:10:51 +00:00
tixie force-pushed a11y/semantic from d7ee04e6b7 to 469d165852 2024-06-07 18:11:12 +00:00 Compare
aeddis approved these changes 2024-06-08 11:14:42 +00:00
quentin removed review request for esther 2024-06-08 18:09:14 +00:00
quentin commented 2024-06-08 18:17:20 +00:00
Owner
Copy link

J'ai lu le rapport d'Adrian : merci pour ce rapport, très intéressant et pertinent ce retour.

Je ne pense pas que Esther passera par là prochainement, je vous propose d'avancer sans son accord : je vais l'appeler la semaine prochaine pour lui parler du site web de manière générale, et je prends la responsabilité de mettre en place les mesures correctives si il venait à y avoir un problème - ce que je ne crois pas pour cette PR ;-)

Je propose de fusionner et mettre en ligne directement cette PR. Si il y a des problèmes de droits d'accès, je rajouterai les droits manquants à Tixie.

J'ai lu le rapport d'Adrian : merci pour ce rapport, très intéressant et pertinent ce retour. Je ne pense pas que Esther passera par là prochainement, je vous propose d'avancer sans son accord : je vais l'appeler la semaine prochaine pour lui parler du site web de manière générale, et je prends la responsabilité de mettre en place les mesures correctives si il venait à y avoir un problème - ce que je ne crois pas pour cette PR ;-) Je propose de fusionner et mettre en ligne directement cette PR. Si il y a des problèmes de droits d'accès, je rajouterai les droits manquants à Tixie.
tixie merged commit 6ae76f5b93 into preprod 2024-06-08 19:08:57 +00:00
tixie commented 2024-06-08 19:21:56 +00:00
Author
Owner
Copy link

Quelques problèmes dans la structure du HTML (des balises mal fermées). Je commit le fix dans les jours à venir avant de merge vers main. Merci la tache "HTML syntax check" de la CI !

Quelques problèmes dans la structure du HTML (des balises mal fermées). Je commit le fix dans les jours à venir avant de merge vers main. Merci la tache "HTML syntax check" de la CI !
Sign in to join this conversation.
Reviewers
No reviewers
adrien
aeddis
Labels
Clear labels   
cat
a11y
Accessibility issues

  
cat
compat
Compatibility

  
cat
content
Rédaction/Édition du contenu

  
cat
perf

  
part
calendar
La représentation ASCII Art des calendriers

No labels
cat
a11y
cat
compat
cat
content
cat
perf
part
calendar
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
tixie
adrien
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Deuxfleurs/site#42
No description provided.
Delete branch "tixie/deuxfleurs-website:a11y/semantic"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?