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

[Deque Analysis] Action/Admin Sidebar is not entirely keyboard accessible #1170

Closed
tdonohue opened this issue May 10, 2021 · 3 comments · Fixed by #1230
Closed

[Deque Analysis] Action/Admin Sidebar is not entirely keyboard accessible #1170

tdonohue opened this issue May 10, 2021 · 3 comments · Fixed by #1230
Assignees
Labels
accessibility bug component: administrative tools Related to the admin menu or tools e/2 Estimate in hours high priority
Milestone

Comments

@tdonohue
Copy link
Member

tdonohue commented May 10, 2021

Critical: This issue results in blocked content for people with disabilities, and will definitely prevent them from accessing fundamental features or content. This type of issue puts your organization at risk. Prioritize fixing as soon as possible, within the week if possible. Remediation should be a top priority.

Deque Analysis Summary

A total of 3 Critical issues exist in the Action/Admin sidebar, all pertaining to keyboard accessibility:

  1. (ID 470074) Under "New" menu, "Community", "Collection" and "Item" link are inaccessible to keyboard. Strangely, when that menu is open, clicking Tab jumps to "Process". (It may be because these three entries use Javascript and have an a tag without an href.)
    • This same issue exists under the "Edit" section, and seemingly for any entry that uses ds-onclick-menu-item
  2. (IDs 470078, 470093) Many links in the sidebar do NOT include an href (which I believe causes issue 1). All links MUST have an href.
    • One way to fix this issue by providing an HREF value (even if JavaScript event handlers are used) for the link. (Note: Ensure that if JavaScript event handlers are used, they allow the link to be activated with the Enter key in addition to mouse click.)
    • e.g. <a href=quickview.html">Quick view</a>
    • e.g. <a href="javascript:void(0);">Quick view</a>

The full list of these issues (which includes links for more info) can be found at this link (requires login): https://axeauditor.dequecloud.com/test-run/0856438a-a19a-11eb-bc31-b7d5be387c86/issues?activeTab=dt-issue&page=0&pageSize=100&sortField=ordinal&sortDir=asc&row=9&filter%5Bseverity%5D=4&filter%5Bpage_number%5D=9

More Information / Tools

@tdonohue
Copy link
Member Author

@artlowel : assigning this to you / your team for analysis.

@tdonohue tdonohue added the component: administrative tools Related to the admin menu or tools label May 10, 2021
@artlowel
Copy link
Member

We'll add hrefs to all links, and ensure the tab order makes sense

This will take an estimated 2 hours and is best done in the context of #1185

@tdonohue
Copy link
Member Author

Sounds good @artlowel . Adding that estimate & assigning back to you for your team. Feel free to assign both tickets to the same individual & one PR is perfectly OK.

@tdonohue tdonohue added e/2 Estimate in hours and removed Estimate TBD labels May 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility bug component: administrative tools Related to the admin menu or tools e/2 Estimate in hours high priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants