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

⌨ (Admin portal) Table Headers on "Classifications" Page Do Not Announce Sort Functionality to Screen Readers #12157

Open
3 tasks
MaddyDaigle opened this issue Nov 29, 2024 · 1 comment · May be fixed by #12453
Assignees
Labels
accessibility Issues related to accessibility

Comments

@MaddyDaigle
Copy link
Contributor

⌨ Accessibility Issue

Table headers do not provide screen reader users with feedback indicating that the columns are sortable. When navigating to the table headers, users are not informed that sorting functionality is available or when sorting actions occur.

WCAG 2.1 Criteria

1.3.1 Info and Relationships: Ensure that relationships, such as sortable headers, are programmatically determined.
4.1.2 Name, Role, Value: Ensure UI components provide the correct name, role, and state to assistive technologies.
2.4.4 Link Purpose (In Context): Ensure that users understand the purpose of interactive elements like sortable headers.

🦋 Expected Behaviour

When a user navigates to a sortable table header, the screen reader should announce:

  • The header’s name.
  • That it is sortable.
  • The current sort state (e.g., "ascending" or "descending").
  • When sorting is activated, the screen reader should announce the new sort state.

🕵️ Details

Operating System: Windows 10
Browser: Edge (latest version at the time of testing)
Assistive Technology: NVDA screen reader (latest version)

📋 Steps to Reproduce

  • Open the "Admin Classifications" page in Edge.
  • Launch NVDA and navigate to the table headers using the keyboard.
  • Attempt to sort a column by activating a header.
  • Note the absence of feedback indicating that sorting is possible or has occurred.

📸 Screenshot

🙋‍♀️ Proposed Solution

Use the aria-sort attribute to indicate the sort state.

✅ Acceptance Criteria

  • NVDA announces that the table headers are sortable when focused.
  • Screen readers announce the new sort state when sorting is triggered.
  • Sortable headers meet WCAG 2.1 standards for labeling and dynamic updates.
@esizer esizer added the accessibility Issues related to accessibility label Nov 29, 2024
@tristan-orourke tristan-orourke moved this to 🏭 Ready for Estimate in GC Digital Talent Dec 16, 2024
@tristan-orourke
Copy link
Member

Sortable headers with no active sort state should indicate they are sortable with the aria-sort attribute.
When you do change sort state, we'll need to announce something if aria-sort doesn't work automatically.
If you need a new bit of copy, a phrase to announce, work directly with @NienkeBr

@tristan-orourke tristan-orourke moved this from 🏭 Ready for Estimate to 📋 Ready for Dev in GC Digital Talent Jan 9, 2025
@esizer esizer self-assigned this Jan 9, 2025
@esizer esizer moved this from 📋 Ready for Dev to 🏗 In progress in GC Digital Talent Jan 9, 2025
@esizer esizer linked a pull request Jan 9, 2025 that will close this issue
@esizer esizer moved this from 🏗 In progress to 👀 In review in GC Digital Talent Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Issues related to accessibility
Projects
Status: 👀 In review
Development

Successfully merging a pull request may close this issue.

3 participants