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

[Accessibility] "All of DSpace" menu in header does not expose the presence of the submenu & its state via ARIA attributes #2699

Closed
tdonohue opened this issue Dec 6, 2023 · 3 comments · Fixed by #2676

Comments

@tdonohue
Copy link
Member

tdonohue commented Dec 6, 2023

Originally reported in #1174 via the Deque Analysis in 2021. Rated "serious" by Deque. This is one level less than the highest "critical" rating, and is described by Deque as follows:

Serious: This issue results in serious barriers for people with disabilities, and will partially prevent them from accessing fundamental features or content. People relying on assistive technologies will experience significant frustration as a result. Issues falling under this category are major problems, and remediation should be a priority.

Describe the bug

(Deque Issue ID: 468467) "All of DSpace" submenu: The presence of the submenu and its state are not exposed.

  • Suggested fixes include BOTH of these:
    • (1) Use aria-haspopup="menu" on the element that toggles the submenu open/closed. AND
    • (2) Use the aria-expanded attribute on the element that toggles the submenu open/closed with a value of "true" (when the submenu is expanded) or "false" (when the submenu is collapsed).

To Reproduce
Steps to reproduce the behavior:

  1. Can be seen on the https://demo.dspace.org site. These ARIA labels are missing and never updated based on the current state of the menu.

Related work
Issue initially reported in #1174

@tdonohue tdonohue added bug help wanted Needs a volunteer to claim to move forward accessibility good first issue This is a smaller ticket/bug that may be easier for new contributors medium priority labels Dec 6, 2023
@CrisGuzmanS
Copy link
Contributor

Hello @tdonohue can i take this issue?

@CrisGuzmanS CrisGuzmanS mentioned this issue Dec 8, 2023
6 tasks
@tdonohue tdonohue moved this from 📋 To Do to 🏗 In Progress in DSpace 8.x and 7.6.x Maintenance Dec 12, 2023
@tdonohue tdonohue removed help wanted Needs a volunteer to claim to move forward good first issue This is a smaller ticket/bug that may be easier for new contributors labels Dec 12, 2023
@tdonohue
Copy link
Member Author

Assigned to @davide-negretti as his larger refactor in #2676 fixes this small issue.

@tdonohue
Copy link
Member Author

Ported to 7.x in #2858

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: ✅ Done
3 participants