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

Attributs sur menu mobile #165

Merged
merged 4 commits into from
Jan 6, 2025
Merged

Attributs sur menu mobile #165

merged 4 commits into from
Jan 6, 2025

Conversation

ffoodd
Copy link
Contributor

@ffoodd ffoodd commented Dec 20, 2024

J’ai entamé la mise en conformité RGAA du menu principal.

Je n’ai pas testé encore, et il y a du ménage à faire :

  • Vérifier que le focus est bien piégé dans le menu ;
  • Vérifier l’utilité de la méthode listenIntersection, qui ne semble jamais appelée ?
  • Peut-être brancher la nouvelle méthode initModalDialog dessus, pour réagir au resize de la page ?
  • Et finalement, compiler tout le bazar et mettre à jour côté site…

@ffoodd ffoodd requested a review from joachimesque December 20, 2024 16:24
@ffoodd ffoodd self-assigned this Dec 20, 2024
@ffoodd ffoodd marked this pull request as ready for review January 6, 2025 14:35
site/layouts/partials/nav.html Outdated Show resolved Hide resolved
site/layouts/partials/template/nav.html Outdated Show resolved Hide resolved
@ffoodd ffoodd merged commit 37c01b9 into master Jan 6, 2025
1 check failed
@ffoodd ffoodd deleted the fix/mobile-nav branch January 6, 2025 19:24
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.

2 participants