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

PFE-Dropdown has incorrect aria and a11y setup #2054

Open
wesruv opened this issue Jul 1, 2022 · 4 comments
Open

PFE-Dropdown has incorrect aria and a11y setup #2054

wesruv opened this issue Jul 1, 2022 · 4 comments
Labels
accessibility Improve accessibility priority: medium Severity level: 2 RHDS

Comments

@wesruv
Copy link
Contributor

wesruv commented Jul 1, 2022

Description of the issue

This was noticed on PFE 1.x, but I suspect it might also be in 2.x, full discussion is here: https://chat.google.com/room/AAAA0siaVdA/V0r-JRHGmOY

It should follow the aria/accessibility setup from: https://pattern-library.dequelabs.com/components/selects

Basically it should be following the listbox pattern, instead of a menu, menu has very specific expectations similar to the "File" menu in applications:

While the term "menu" is generically used a term used to describe site navigation, the menu role is for a list of action or functions that require complex functionality, such as composite widget focus management and first-character navigation
From: https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/menu_role

Impacted component(s)

  • pfe-dropdown
@wesruv wesruv added accessibility Improve accessibility priority: medium Severity level: 2 RHDS labels Jul 1, 2022
@wesruv
Copy link
Contributor Author

wesruv commented Jul 1, 2022

I don't see this component in RHDS yet, adding the tag because I'm guessing it might be headed there?

@bennypowers
Copy link
Member

bennypowers commented Jul 8, 2022

The next step for pfe-dropdown will be to align it with PFv4. We could address this during that process. Is it the case that you will want dropdown in RHDS, being the successor to pfe 1.x dropdown? in that case please fill out a request for new component there.

@wesruv
Copy link
Contributor Author

wesruv commented Jul 8, 2022

Cool, I think it'd be great to have this in RHDS, I'd guess @kylebuch8 && @TylerMartin5 would also be interested. Made a ticket: RedHat-UX/red-hat-design-system#425

@bennypowers
Copy link
Member

See also #2049

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Improve accessibility priority: medium Severity level: 2 RHDS
Projects
None yet
Development

No branches or pull requests

2 participants