-
Notifications
You must be signed in to change notification settings - Fork 232
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
Example aria role status #431
Conversation
Provided alert demo for shopping cart.
Since there may be multiple working examples, moved first sample from index.html to a unique name
changing from role of alert to role of status. Also added in default values as well as more explanatory paragraph
Not sure I got the right number of hierarchical jumps right, but the links to the technique pages at least contain a valid target file.
Moved to new location as per MC's instructions
Rawgit link to the technique: https://rawgit.com/w3c/wcag/tech-using-role-status/techniques/aria/aria-status-role.html Rawgit link to working example - search results https://rawgit.com/w3c/wcag/example-aria-role-status/working-examples/example-aria-role-status/searchresults.html Rawgit link to working example - shopping cart https://rawgit.com/w3c/wcag/example-aria-role-status/working-examples/example-aria-role-status/shoppingcart.html |
picked up numbering from existing one
Minor edits to fix some validator issues.
removed extraneous line
minor clean-up and a tweak to the counter script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made a few tweaks to the sample code, but both samples worked for me using iOS and VO.
From the call 2018-08-07 this had 5 x +1s. |
removed explicit setting of deafult values
Removing aria-live and aria-atomic means (in my test page, http://3needs.org/en/testing/code/role-status.html , Example 2) that now, the context is no longer spoken in Firefox/NVDA - arguably a very important UA/AT combo. |
Tech using role status
Submitting this pull request, along with the technique for aria-status-role, which I will cross-reference.