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

WCAG status messages when update filter #1244

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

BarbaraOliveira13
Copy link

@BarbaraOliveira13 BarbaraOliveira13 commented Jan 20, 2025

🎩 What? Why?

Adds aria-live="polite" and aria-atomic="true" attributes to the h2 element displaying the number of results on the components index page: proposals, debates, meetings & sortitions. These attributes ensure that the updates to the number of results, are announced by screen readers when filters are applied.

🎩 What? Why?

This change addresses accessibility issues related to WCAG 4.1.3 - Status Messages. It ensures that screen readers announce the updated results count dynamically.

📌 Related Issues

Grist => I manage to add the changes, in the others components related: debates, meetings & sortitions
WCAG 4.1.3 - Status Messages

📋 Steps to Test

  1. Navigate to the proposals, debates, meetings or sortitions index page (e.g., /processes/budget-participatif/f/1/proposals).
  2. Apply filters.
  3. Verify with the inspector:
  • aria-live="polite" and aria-atomic="true" are present in the h2 element

📷 Screenshots (optional)

Capture d’écran 2025-01-20 à 15 57 55

Additional context
This issue was identified during an accessibility audit targeting compliance with RGAA and WCAG standards. The change ensures that users relying on assistive technologies are informed of updates in real time when filters are applied.

@BarbaraOliveira13 BarbaraOliveira13 marked this pull request as ready for review January 20, 2025 15:49
@BarbaraOliveira13 BarbaraOliveira13 force-pushed the fix/WCAG-status-messages-when-update-filter branch from dd0a6e3 to 725b120 Compare February 17, 2025 17:17
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