-
Notifications
You must be signed in to change notification settings - Fork 60
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
fix JAWS reading dialog title when closing dropdown widget #1808
Conversation
8904e59
to
b559305
Compare
…n widget This commit contains a better fix for the problem described in ariatemplates#1797. When a dropdown widget with waiAria:true is inside a dialog with role=dialog the dropdown DOM element is no longer appended to the <body> but to the element that has role=dialog. Therefore, screen readers correctly understand that the dropdown is not outside the dialog and no longer read the dialog title when changing the focus after closing the dropdown. PTR 14068536
…n widget This commit contains a better fix for the problem described in ariatemplates#1797. When a dropdown widget with waiAria:true is inside a dialog with role=dialog the dropdown DOM element is no longer appended to the <body> but to the element that has role=dialog. Therefore, screen readers correctly understand that the dropdown is not outside the dialog and no longer read the dialog title when changing the focus after closing the dropdown. PTR 14068536
b559305
to
4810d99
Compare
Codecov Report
@@ Coverage Diff @@
## master #1808 +/- ##
==========================================
- Coverage 81.02% 80.99% -0.03%
==========================================
Files 599 600 +1
Lines 33650 33662 +12
==========================================
+ Hits 27264 27265 +1
- Misses 6386 6397 +11
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks good to me! :)
@ymeine Thank you for your review! |
This commit contains a better fix for the problem described in #1797. When a dropdown widget with waiAria:true is inside a dialog with role=dialog the dropdown DOM element is no longer appended to the <body> but to the element that has role=dialog. Therefore, screen readers correctly understand that the dropdown is not outside the dialog and no longer read the dialog title when changing the focus after closing the dropdown. PTR 14068536 (cherry picked from commit 40b097e)
This PR contains a better fix for the problem described in #1797.
When a dropdown widget with
waiAria:true
is inside a dialog withrole=dialog
the dropdown DOM element is no longer appended to the<body>
but to the element that hasrole=dialog
. Therefore, screen readers correctly understand that the dropdown is not outside the dialog and no longer read the dialog title when changing the focus after closing the dropdown.