Skip to content

Conversation

@Stef-Rousset
Copy link

🎩 What? Why?

This PR replaces in the breadcrumb_menu meaningless div and span by p, to improve accessibility.

This PR comes from the audit of Angers city (page 37) and refers to criteria 1.3.1 from WCAG.

📌 Related Issues

Testing

  1. Go to the processes index page
  2. Activate your screen reader, and navigate to the breadcrumb menu until you get to the comments (see screenshot)
  3. Ensure that the time (1 day ago) and the title (New comment) are identified as text by the screen reader.
  4. Open your dev tools and see that they are displayed in a p element.

📷 Screenshots (optional)

Capture d’écran 2025-08-29 à 09 11 10

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