Close current tab button is missing an accessible name and role
Categories
(Firefox :: Tabbed Browser, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox131 | --- | fixed |
People
(Reporter: ayeddi, Assigned: ayeddi)
References
(Blocks 1 open bug)
Details
(Keywords: access)
Attachments
(3 files)
The tab-close-button is not labeled and is missing an interactive role of button, while it is functioning as one.
Note: we do not want this control to be keyboard focusable, because keyboard-only user could close the tab via the context menu and we don't want to create an additional tab stop for the navigation as well, but making sure the control is marked up as a button with an accessible name would allow it to be actionable with speech-to-text software, with touch devices, with switch controls in scan mode, and for screen readers via their navigation shortcuts as well.
Assignee | ||
Comment 1•5 months ago
|
||
The tab-close-button is not labeled and is missing an interactive role of button, while it is functioning as one.
Note: we do not want this control to be keyboard focusable, because keyboard-only user could close the tab via the context menu and we don't want to create an additional tab stop for the navigation as well, but making sure the control is marked up as a button with an accessible name would allow it to be actionable with speech-to-text software, with touch devices, with switch controls in scan mode, and for screen readers via their navigation shortcuts as well.
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/87d3cf2b45b2 Close current tab button is missing an accessible name and role. r=tabbrowser-reviewers,fluent-reviewers,dao,bolsson,flod
Comment 3•2 months ago
|
||
Backed out for causing bc failures in browser_multiselect_tabs_close.js
- Backout link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | browser/components/tabbrowser/test/browser/tabs/browser_multiselect_tabs_close.js | Close tab button has an expected accessible name with 1 tabs (multi) selected. - {"args":{"tabCount":1},"id":"tabbrowser-close-tabs-button"} deepEqual {"id":"tabbrowser-close-tabs-tooltip","args":{"tabCount":1}} -
Assignee | ||
Comment 4•2 months ago
|
||
Thank you, Cristian! Pushed the fixed test now.
Pushed by ayeddi@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6ac47dbd939c Close current tab button is missing an accessible name and role. r=tabbrowser-reviewers,fluent-reviewers,dao,bolsson,flod
Comment 6•2 months ago
|
||
Backed out for causing bc failures in browser_tab_tooltips.js
- Backout link
- Push with failures
- Failure Log
- Failure line: TEST-UNEXPECTED-FAIL | browser/components/tabbrowser/test/browser/tabs/browser_tab_tooltips.js | Test timed out -
Comment 7•2 months ago
|
||
Assignee | ||
Comment 8•2 months ago
|
||
Oh, sure. I’ll fix it today: thank you for the investigation
Assignee | ||
Comment 9•1 month ago
|
||
removing the hanging NI since there is one of test failures that I do not know yet how to explain and will need a help from someone in Layout team, I think.
Comment 11•1 month ago
|
||
So browser_tab_tooltips.js
expects that the tooltip element with the id tabbrowser-tab-tooltip
will be used for the close button, but that won't be the case anymore with this patch. Instead an anonymous tooltip node will be used. You can get that node like this:
document.addEventListener("popupshowing", event => {
// event.target is the root node, event.originalTarget is the anonymous tooltip node
});
Does this help?
Comment 12•28 days ago
|
||
Here's how I thought browser_tab_tooltips.js
should be modified to deal with your changes. So far I've only run it without your changes though (and it passed).
Comment 13•14 days ago
|
||
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/98f09025ffa6 Close current tab button is missing an accessible name and role. r=tabbrowser-reviewers,fluent-reviewers,dao,bolsson,flod
Comment 14•13 days ago
|
||
bugherder |
Updated•13 days ago
|
Description
•