Closed
Bug 1537647
Opened 6 years ago
Closed 6 years ago
Focus goes to limbo when Firefox Accounts toolbar button is activated from keyboard
Categories
(Firefox :: Keyboard Navigation, defect, P1)
Firefox
Keyboard Navigation
Tracking
()
VERIFIED
FIXED
Firefox 68
People
(Reporter: Jamie, Assigned: vbudhram)
References
Details
(Keywords: access)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
STR:
- Press control+l to focus the Location bar.
- Tab to the group of toolbar buttons containing the Firefox Accounts button and then press right arrow until Firefox Accounts is focused.
- Press space to activate it.
- Expected: The first button inside the Firefox Accounts panel should get focus, as is the case for the main Firefox menu button, the Library button, etc.
- Actual: Focus goes nowhere.
Bug 1536514 fixed things so that this button can be activated from the keyboard. However, focus still goes to limbo when it is pressed. We apparently don't want focus to go somewhere for mouse users, but for keyboard users (especially screen reader users), focus shouldn't go to limbo.
This should be a trivial fix. In gSync.toggleAccountPanel, we should just be able to pass the event to PanelUI.showSubView:
https://searchfox.org/mozilla-central/source/browser/base/content/browser-sync.js#292
That is, this should be:
PanelUI.showSubView(viewId, anchor, aEvent);
We should also add a test to browser/base/content/test/keyboard/browser_toolbarButtonKeyPress.js.
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → vbudhram
Updated•6 years ago
|
Priority: -- → P1
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/5fcf07f6cc74
fix focus on FxA menu when activated via keyboard r=Gijs
Comment 3•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox68:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
Reporter | ||
Comment 5•6 years ago
|
||
Verified fixed in Firefox 68.0a1 build 20190324214519.
Status: RESOLVED → VERIFIED
Flags: needinfo?(jteh)
Updated•6 years ago
|
Comment 6•6 years ago
|
||
bugherder uplift |
status-firefox67:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•