-
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
fix: SPA page loading and accessibility #522
base: main
Are you sure you want to change the base?
Conversation
Add live region component Add title to routes Add vueuse package Set focus to skip link on nav Set page title to live region on nav
✅ Deploy Preview for ecospheres ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for meteo-france ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Adapt title generation for custom titles Better behaviour for skip links Custom routes titles
Après tests lecteurs d'écran: ça fonctionne bien 😊 Les liens d'évitements ne seront visibles qu'à la navigation clavier mais tout le temps focus au chagement de page par défaut. Ça évite de les afficher en permanence pour les utilisateurs non concernés. Je me demande s'il ne faudrait pas sortir le fil d'Ariane du La structure mise en place se base sur les recherches utilisateurs effectuées par Marcy Sutton.
Seul truc un peu bizarre: NVDA sur firefox annonce deux fois le nom de la page. C'est à priori un bug connu du logiciel. J'ai rencontré un petit casse-tête avec la page de détails des bouquets qui utilise J'ai laissé la zone d'annonce des titres visible pour que vous puissiez tester sans lecteur d'écran 😅 |
Fix double nav on secondary menu
Fix ecolabdata/ecospheres#334
Fix ecolabdata/ecospheres#340