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

H5P listings page: visible label and accessible name of buttons do not match #1302

Open
MichSpieler opened this issue Dec 14, 2023 · 3 comments
Assignees

Comments

@MichSpieler
Copy link

MichSpieler commented Dec 14, 2023

Description

A network manager has reported that he H5P listing pages the visible label and accessible name do not match on any of the buttons. The buttons just say “Show Activity” and the labels say “View H5P activity.”
image
image
image
Button text by default: Expand All
Button text when expanded: Hide All
aria-label: Expand all activities

image
image
Button text by default: Show Activity
Button text when expanded: Hide activity
aria-label: View h5p activity

Steps to Reproduce

  1. Navigate to a book's h5p-listings page
  2. Inspect buttons
  3. Click buttons and inspect again.

Expected behavior:

  • When the button says "Expand All", the aria-label should say "Expand All"; when the button says "Hide All", the aria-label should say "Hide All"
  • When the button says "Show Activity", the aria-label should say "Show Activity"; when the button says "Hide Activity", the aria-label should say "Hide Activity"

Actual behavior:

  • When the button says "Expand All", the aria-label says "Expand all activities"; when the button says "Hide All", the aria-label says "Expand all activities"
  • When the button says "Show Activity", the aria-label says "View h5p activity"
@MichSpieler
Copy link
Author

For internal reference: Ticket #

@MichSpieler
Copy link
Author

The fact that the visible label and the accessible name do not match was also commented on in a recent accessibility review by a network manager (link accessible for PB Support staff only).

@SteelWagstaff
Copy link
Member

@cmurtagh I'd be happy to open a PR addressing this if the team is open to it?

@SteelWagstaff SteelWagstaff self-assigned this Jan 3, 2025
@SteelWagstaff SteelWagstaff transferred this issue from pressbooks/pressbooks Jan 3, 2025
@SteelWagstaff SteelWagstaff assigned arzola and unassigned cmurtagh and SteelWagstaff Jan 3, 2025
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

No branches or pull requests

4 participants