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

Optimization (accessibility): Elements with an ARIA [role] that require children #4184

Closed
rfultz opened this issue Nov 5, 2020 · 0 comments · Fixed by #4544
Closed

Optimization (accessibility): Elements with an ARIA [role] that require children #4184

rfultz opened this issue Nov 5, 2020 · 0 comments · Fixed by #4544

Comments

@rfultz
Copy link
Contributor

rfultz commented Nov 5, 2020

Background

Looking to optimize the site's performance and that of the homepage specifically, the Lighthouse tool in Chromium browsers has made some suggestions. Some of these will have repercussions throughout the app so we're going to make them their own tickets.

Accessibility Recommendation: Elements with an ARIA [role] that require children to contain a specific [role] are missing some or all of those required children

Some ARIA parent roles must contain specific child roles to perform their intended accessibility functions. details

Overview

The WCCF widget needs its aria-role values fixed, specifically the required parent-child relationships. ("row" doesn't have to be in a table but it is required to have certain children)

Benefit (the 80 of 80/20)

Will help aria readers access table data

Effort (the 20 of 80/20)

Minimal

Complications

  • Unknown
  • Make sure major browsers aren't negatively impacted by any of these changes (Chrome, Safari, Firefox. IE?)

Related tickets

This was referenced Nov 5, 2020
@rfultz rfultz added this to the Sprint 14.3 milestone Mar 17, 2021
@rfultz rfultz modified the milestones: Sprint 14.3, Sprint 14.4 Apr 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants