Add an exception from a11y_checks for clicks on a non-interactive header containers in Add-On tests
Categories
(Toolkit :: Add-ons Manager, task, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox123 | --- | fixed |
People
(Reporter: ayeddi, Assigned: ayeddi)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(1 file)
The toolkit/mozapps/extensions/test/browser/browser_addon_list_reordering.js
sends a click on the static h1.header-name
element as well as (browser_html_list_view.js)[https://searchfox.org/mozilla-central/rev/9c509b8feb28c1e76ad41e65bf9fd87ef672b00f/toolkit/mozapps/extensions/test/browser/browser_html_list_view.js#400-405,938,975] does in multiple places. These tests are purposefully clicking on this non-interactive text outside of the tested list to clear any focus.
For users of keyboards and assistive technology another way like pressing Esc
key is available to dismiss the dialog, thus we need to add an exception from the a11y_checks for this tests via setEnv
and remove the fail-if
notation from its test manifest.
Assignee | ||
Comment 1•5 months ago
|
||
The toolkit/mozapps/extensions/test/browser/browser_addon_list_reordering.js
sends a click on the static h1.header-name
element as well as (browser_html_list_view.js
)[https://searchfox.org/mozilla-central/rev/9c509b8feb28c1e76ad41e65bf9fd87ef672b00f/toolkit/mozapps/extensions/test/browser/browser_html_list_view.js#400-405] does. These tests are purposefully clicking on this non-interactive text outside of the tested list to clear any focus.
For users of keyboards and assistive technology another way like pressing Esc
key is available to dismiss the dialog, thus we need to add an exception from the a11y_checks for this tests via setEnv
and remove/update the fail-if
notation in its test manifest.
Updated•5 months ago
|
Assignee | ||
Updated•5 months ago
|
Updated•5 months ago
|
Pushed by ayeddi@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2d2b0325ced3 Add an exception from a11y_checks for clicks on a non-interactive h1.header-name in Add-Ons tests. r=rpl
Comment 3•5 months ago
|
||
bugherder |
Description
•