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

Review 1.2 IDL(Interface Definition Language) button example #727

Closed
mcking65 opened this issue Jun 20, 2018 · 6 comments
Closed

Review 1.2 IDL(Interface Definition Language) button example #727

mcking65 opened this issue Jun 20, 2018 · 6 comments
Assignees
Labels
Example Page Related to a page containing an example implementation of a pattern

Comments

@mcking65
Copy link
Contributor

The APG 1.2 IDL Button Example developed for issue #701 is ready for task force review.

@annabbott
Copy link

annabbott commented Jun 20, 2018

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:
Mute button does not function with either Space or Enter. Only NVDA announced Mute button "not pressed".

With JAWS 2018.1805.33 and NVDA 2018.1.1 in IE11 on Win10:
Mute button does not function with either Space or Enter. Pressed/not pressed state not announced. Graphic on button to indicate Mute/Unmute does not appear.

@mcking65 mcking65 added Example Page Related to a page containing an example implementation of a pattern Needs Review labels Jun 20, 2018
@mcking65 mcking65 added this to the 1.2 APG Working Draft 1 milestone Jun 20, 2018
@jnurthen
Copy link
Member

In order to test this you need to run Chrome Canary and use the --enable-blink-features=AccessibilityObjectModel flag.

@mcking65 mcking65 modified the milestones: 1.2 APG Working Draft 1, 1.2 APG Working Draft 2 Jun 25, 2018
@mcking65 mcking65 modified the milestones: 1.2 APG Second Working Draft, 1.2 Third Working Draft Oct 1, 2018
@scottaohara
Copy link
Member

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.
The SVG changes when using space or enter to toggle the button, though aria-pressed does not change. It doesn't matter if i've turned on the AOM flag in firefox or not.

@a11ydoer a11ydoer changed the title Review 1.2 IDL button example Review 1.2 IDL(Interface Definition Language) button example Aug 16, 2019
@carmacleod carmacleod self-assigned this Aug 27, 2019
@carmacleod
Copy link
Contributor

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)
Also, use createElement instead of having the element already in the DOM.

@carmacleod
Copy link
Contributor

carmacleod commented Aug 28, 2019

@mcking65
The code for the "Button" and "Button IDL" examples has diverged a fair bit due to updates to the Button example on Sept 13, 2018, which were after the "Button IDL" code was added on June 19, 2018.

Since the IDL example states that:

The JavaScript for the examples on this page uses the IDL interface defined in ARIA 1.2. In all other respects, these examples are identical to the Button Examples.

... 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).
i.e. I would think that "Mute button Pressed" or "Mute button Not Pressed" would be preferred over "Mute button Pressed Mute" or "Mute button Not Pressed Mute"?

mcking65 pushed a commit that referenced this issue Sep 27, 2019
… (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.
@mcking65
Copy link
Contributor Author

Resolved with commit 3d83acd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Example Page Related to a page containing an example implementation of a pattern
Development

No branches or pull requests

5 participants