Skip to content

Conversation

@Stef-Rousset
Copy link

@Stef-Rousset Stef-Rousset commented Aug 18, 2025

🎩 What? Why?

This PR improves accessibility in header page by adding role attribute on div.main-bar__search, and by changing div.main-bar__links-desktop for nav.main-bar__links-desktop with role="navigation"

This PR comes from the audit of Angers city (page 22) and refers on criteria 1.3.1 from WCAG

📌 Related Issues

Testing

  1. As a user, go to home page and open dev tools
  2. Check in the header that the search div has a role="search" and that the links are in a nav element with role="navigation"

📷 Screenshots (optional)

Capture d’écran 2025-08-18 à 15 45 00

@Stef-Rousset Stef-Rousset changed the title fix: add missing roles on div in header Fix: improve accessibility in header Aug 18, 2025
@Stef-Rousset Stef-Rousset changed the title Fix: improve accessibility in header Fix accessibility in header improvement Aug 18, 2025
@stale
Copy link

stale bot commented Oct 18, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. @carolromero & @xabier feel free to chime in.

@stale stale bot added the wontfix under the radar label Oct 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants