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 aria-modal notes added to dialog example page #503

Closed
mcking65 opened this issue Oct 28, 2017 · 4 comments
Closed

Review aria-modal notes added to dialog example page #503

mcking65 opened this issue Oct 28, 2017 · 4 comments
Assignees
Labels
Example Page Related to a page containing an example implementation of a pattern

Comments

@mcking65
Copy link
Contributor

For issue #480, added some notes following the roles, states, and properties table on the
modal dialog example page.

Please review for editorial issues.

@mcking65 mcking65 added Example Page Related to a page containing an example implementation of a pattern Needs Review labels Oct 28, 2017
@mcking65 mcking65 added this to the 1.1 APG Release 1 milestone Oct 28, 2017
@annabbott
Copy link

annabbott commented Oct 30, 2017

For "End of the Road!" dialog > using FF 56.0.2 with JAWS 18.0.4350 on Win10:
TAB key only goes to the first Close button and the buttons at the bottom of the dialog cannot be reached via keyboard. It is confusing for an example to not place focus via the TAB key on what appears to be functional buttons. Suggest, at a minimum, those buttons at bottom of dialog should display as disabled buttons.

Same results using Chrome Version 62.0.3202.75 (Official Build) (64-bit) with JAWS 18.0.4350 on Win10.

@annabbott
Copy link

annabbott commented Oct 30, 2017

Address Added dialog > using FF 56.0.2 with JAWS 18.0.4350 on Win10:
JAWS announces paragraph but, on initial pass, does not identify "your profile" as a link (does announce "OK" as a button). When I Shift-TAB back to "your profile", it is announced as a link. Unsure whether this is a code bug or a JAWS bug, however for a JAWS user this would be very confusing.

Same results using Chrome Version 62.0.3202.75 (Official Build) (64-bit) with JAWS 18.0.4350 on Win10.

@annabbott
Copy link

I don't see any issues in the text following the roles, states, and properties table.

@mcking65
Copy link
Contributor Author

mcking65 commented Nov 16, 2017

@annabbott commented:

For "End of the Road!" dialog > using FF 56.0.2 with JAWS 18.0.4350 on Win10:
TAB key only goes to the first Close button and the buttons at the bottom of the dialog cannot be reached via keyboard.
It is confusing for an example to not place focus via the TAB key on what appears to be functional buttons.
Suggest, at a minimum, those buttons at bottom of dialog should display as disabled buttons.

Ann, those buttons are in a dialog that is under the "end of the road" dialog. That entire underlying dialog should be grayed out. I raised issue 523 to address that.

Address Added dialog > using FF 56.0.2 with JAWS 18.0.4350 on Win10:
JAWS announces paragraph but, on initial pass, does not identify "your profile" as a link (does announce "OK" as a button).
When I Shift-TAB back to "your profile", it is announced as a link.
Unsure whether this is a code bug or a JAWS bug, however for a JAWS user this would be very confusing.

This is not a bug. When the dialog opens, most screen readers simply read out the accessible name and description for the dialog. The name and description are fed to the screen reader as flat strings. Those strings cannot contain semantic information like which part of the descriptive information is a link.

Since IA2 provides an accessible relation for content referenced by aria-describedby that is present in the DOM, JAWS and NVDA could potentially utilize that relationship and read directly from the describing content rather than spitting out the flat string. That would open up the possibility of reading it with semantics. I am guessing that would be a nontrivial change. Although, if feasible, it would definitely be a good one for them to make.

Ann, thank you for the thorough review! We have achieved the objective of this issue. So, closing it.

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

2 participants