-
Notifications
You must be signed in to change notification settings - Fork 346
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
Review 1.2 IDL(Interface Definition Language) button example #727
Comments
No text issues noted in APG 1.2 IDL Button Example. However, with JAWS 2018.1805.33 and NVDA 2018.1.1 in Firefox 60.0.2 and Chrome 67.0.3396.87 on Win10: With JAWS 2018.1805.33 and NVDA 2018.1.1 in IE11 on Win10: |
In order to test this you need to run Chrome Canary and use the --enable-blink-features=AccessibilityObjectModel flag. |
Testing this today with Firefox, Safari and Chrome on macOS. Both buttons work as expected with Safari and Chrome with VoiceOver on or off. The mute button does not change state when mouse clicked with Firefox. |
Add more words to the example's description to explain that this is the simplest example of using the IDL and it is just for illustrative purposes. (the example uses element.role and element.ariaPressed in the js) |
@mcking65 Since the IDL example states that:
... the easiest thing for me is to start with a fresh copy of the (updated) button example and just make the IDL-related changes that we discussed (so that the files will only differ by about a dozen lines or so in total). Sound good? Also, just curious why aria-hidden was removed from the "Mute" SVG in the toggle button? Seems to me that the icon is redundant (i.e. because the button text is also "Mute") so it would be better if the SVG was aria-hidden, particularly as JAWS only says "Mute" and doesn't differentiate between the 2 forms of the icon (speaker with an X through it vs. speaker with sound waves in front of it). |
… (pull #1161) Closes #727 by making the following changes: * Revise introduction to clarify purpose of the example. * Remove "similar examples" list. * Set role and aria-pressed in init() method instead of staticly in HTML. * add comments in JS documenting the equivalent way to code without reflection. * Add aria-hidden and focusable==false" to the svg.
Resolved with commit 3d83acd. |
The APG 1.2 IDL Button Example developed for issue #701 is ready for task force review.
The text was updated successfully, but these errors were encountered: