-
Notifications
You must be signed in to change notification settings - Fork 31
Refactored Test plan for The Disclosure of Answers to Frequently Asked Questions Example #1237
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
base: master
Are you sure you want to change the base?
Conversation
…s. Here we test the fourth disclosure as well
Made the assertion: "List boundary is conveyed". The description list boundary is technically more correct. However, the ARIA mapping is actually "association list" and every screen reader is handling this differently. There has been discussion in ARIA about this mapping. No one seems to be happy with it. I am not confident it would be a good idea to fail screen readers that simply call it a list. That is what NVDA and Narrator do. JAWS still calls it a "definition list". VoiceOver calls it a description list. Open to further discussion about this change.
* Change button reference to htmlAam type. * Added a dd reference * Added references to assertions.
For consistency with prior plans and editorial guidelines, changed parentheses to single quotes so assertions of the form: >Change in state (to expanded) is conveyed are now: >Change in state to 'expanded' is conveyed
Somehow there are 4859 files changed by this pull request. I think you might have committed a temp working directory or something like that. I can't get the files tab of this PR to function with that many files changed. |
I pushed commits with the following changes ... Changed 'DescriptionListBoundary' to "listBoundary'Made the assertion: "List boundary is conveyed". The description list boundary is technically more correct. However, the ARIA mapping is actually "association list" and every screen reader is handling this differently. There has been discussion in ARIA about this mapping. No one seems to be happy with it. I am not confident it would be a good idea to fail screen readers that simply call it a list. That is what NVDA and Narrator do. JAWS still calls it a "definition list". VoiceOver calls it a description list. Open to further discussion about this change. Updated references
Editorial changes to state assertionsFor consistency with prior plans and editorial guidelines, changed parentheses to single quotes so assertions of the form:
are now:
|
Thank you @mcking65 |
Preview Tests
Test Plan Updates:
setupScripts
: IntroducedsetupScripts
to programmatically focus and expand the fourth disclosure button.dl
) boundary.