-
Notifications
You must be signed in to change notification settings - Fork 6
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
listbox roles tests #15
Comments
to assign to @rahimabdi |
Please see web-platform-tests/wpt#41324. Is this along the lines of what's expected? |
Hadn't seen this, you can disregard my PR then |
FYI, this would constitute updates to an existing file: ./wpt/wai-aria/role/list-roles.html Add other variants of lists (one simple list is already included) and make sure the roles are computed as expected. |
Sorry @howard-e. We crossed in transit I guess. I didn't even see your replies until I posted the prior comment. (I should have reloaded first.) Let's use your PR, but please move your variants (and add more) to the existing |
Actually, you're right that this should be a new file (listbox-roles.html, not combining with list-roles.html) but please move it to the roles dir with the others. I think this is almost ready to go as-is, but optionally, verify it covers any other listbox patterns from the APG? |
Done! web-platform-tests/wpt#41324 should now be ready for your review |
listbox/option etc.
follow-on from #8
The text was updated successfully, but these errors were encountered: