Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Licence explanation popover contents aren't read to screen reader users #987

Closed
zackkrida opened this issue Feb 22, 2022 · 0 comments · Fixed by #1398
Closed

Licence explanation popover contents aren't read to screen reader users #987

zackkrida opened this issue Feb 22, 2022 · 0 comments · Fixed by #1398
Assignees
@sarayourfriend
Labels
♿️ aspect: a11y Concerns related to the project's accessibility 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents
Milestone
v3.2.1 - Must haves

Comments

@zackkrida
Copy link
Member

This issue is applicable once #850 is merged.

Problem

After selecting one of the license explanation buttons the user is shown the license explanation but the popover contents are not read to screen reader users.

This can be observed in this video (sound on 🔉):

CleanShot.2022-02-22.at.11.22.00-converted.mp4
@zackkrida zackkrida added 🟧 priority: high Stalls work on the project or its dependents ✨ goal: improvement Improvement to an existing user-facing feature 🛠 goal: fix Bug fix ♿️ aspect: a11y Concerns related to the project's accessibility labels Feb 22, 2022
@sarayourfriend sarayourfriend added this to the v3.2.1 - Must haves milestone Apr 26, 2022
@sarayourfriend sarayourfriend removed the ✨ goal: improvement Improvement to an existing user-facing feature label May 9, 2022
@sarayourfriend sarayourfriend self-assigned this May 9, 2022
@sarayourfriend sarayourfriend mentioned this issue May 9, 2022
Merged
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sarayourfriend sarayourfriend

Labels
♿️ aspect: a11y Concerns related to the project's accessibility 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents
Projects
None yet
Milestone
v3.2.1 - Must haves
Development

Successfully merging a pull request may close this issue.

Add license explanation to close button label WordPress/openverse-frontend
2 participants
@zackkrida @sarayourfriend