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

datalist doesn't actually resepct/expose aria-* attributes #382

Closed
scottaohara opened this issue Dec 4, 2021 · 2 comments · Fixed by #383
Closed

datalist doesn't actually resepct/expose aria-* attributes #382

scottaohara opened this issue Dec 4, 2021 · 2 comments · Fixed by #383

Comments

@scottaohara
Copy link
Member

Presently the datalist allows for global aria-* attributes and any aria-* attributes applicable to the listbox role.

In reality no aria-* attributes are respected on the datalist since its rendered ui is completely controlled by the browser.

The datalist attribute allowances should be updated to reflect this reality and simply indicate no ARIA attributes are allowed on this element.

@scottaohara scottaohara added this to the ARIA in HTML: Next milestone Dec 4, 2021
scottaohara added a commit that referenced this issue Dec 4, 2021
closes #382
Removes the allowances of `aria-*` attributes from `datalist` as they are not actually respected / exposed to users.

---

Need two conformance checkers to accept or indicate commitment to this change before it can be merged:

- [ ] [html validator (need to file issue)](https://github.com/validator/validator/issues/)
- [ ] [ibm equal access accessibility checker (need to file issue)](https://github.com/IBMa/equal-access/issues/)
- [ ] [axe-core (need to file issue)](https://github.com/dequelabs/axe-core/issues)
- [ ] [arc toolkit (need to file issue)](https://github.com/ThePacielloGroup/WAI-ARIA-Usage/issues)
@carmacleod
Copy link
Contributor

Will it still have the implicit role of listbox?

@scottaohara
Copy link
Member Author

yes. that's the implicit role for the datalist.

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