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

IA2 live regions: Ignore content marked with aria-live="off" inside a live region #4405

Closed
nvaccessAuto opened this issue Aug 20, 2014 · 6 comments
Labels
Milestone

Comments

@nvaccessAuto
Copy link

@nvaccessAuto nvaccessAuto commented Aug 20, 2014

Reported by jteh on 2014-08-20 06:44
Even though we do check the value of aria-live when determining whether to handle an update, we don't check for aria-live="off" while actually gathering content. aria-live="off" on a node should cause anything inside it to be ignored.

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Aug 21, 2014

Attachment t4405.html added by jteh on 2014-08-21 01:28
Description:
Test case.

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Aug 21, 2014

Comment 1 by jteh on 2014-08-21 01:29
Test case attached. When you press the button, "This added text should not be presented" should not be spoken.

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Aug 22, 2014

Comment 2 by Michael Curran <mick@... on 2014-08-22 02:12
In [07c195f]:

Merge branch 't4405' into next. Incubates #4405

Changes:
Added labels: incubating

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Sep 10, 2014

Comment 3 by Michael Curran <mick@... on 2014-09-10 06:04
In [332af99]:

Merge branch 't4405' into next. Incubates #4405

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Oct 8, 2014

Comment 4 by James Teh <jamie@... on 2014-10-08 00:35
In [af3a512]:

Inside a live region, content which is marked as not being live (e.g. aria-live="off") is now correctly ignored.

Fixes #4405.

Changes:
Removed labels: incubating
State: closed

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Oct 8, 2014

Comment 5 by jteh on 2014-10-08 00:44
Changes:
Milestone changed from next to 2014.4

@nvaccessAuto nvaccessAuto added this to the 2014.4 milestone Nov 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant