Fix down/up arrow behavior with multiple selectable elements per row
Categories
(Firefox :: Address Bar, defect, P2)
Tracking
()
People
(Reporter: dao, Assigned: dao)
References
(Blocks 1 open bug)
Details
(Whiteboard: [snt-urlbar-result-menu])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
From https://phabricator.services.mozilla.com/D161776:
while pressing down to go down the list of results, it focuses on each meatball menu button. That could mess up users' muscle memory. I recognize that the placeholder is shown on each result because the logic of showing/adding the button on a result will come in future patches, but while testing this I found it a bit jarring that I had to press down extra times to select a non-heuristic result.
I agree with jteow. I think we discussed that down/up arrow keys should skip this button while tab should not.
Updated•2 years ago
|
Comment 1•2 years ago
|
||
I agree with having up/down always go through results, while tab moves through "widgets" inside results.
The only special case may be quick actions, but I think we didn't find the right compromise there yet.
Comment 2•2 years ago
|
||
It may be a good idea to coordinate with UX on the best solution here.
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Comment 3•2 years ago
|
||
I just wanted to ask / clarify in reply to Marco's point that we may not want to do this for quickactions. I think based on feedback we almost certainly want this behaviour for QuickActions, in that the down arrow would select the first Action when present then move on to the next UrlbarResult as opposed to the next action and tab would be used to select any further buttons / actions.
Comment 4•2 years ago
|
||
(In reply to Dale Harvey (:daleharvey) from comment #3)
I just wanted to ask / clarify in reply to Marco's point that we may not want to do this for quickactions. I think based on feedback we almost certainly want this behaviour for QuickActions, in that the down arrow would select the first Action when present then move on to the next UrlbarResult as opposed to the next action and tab would be used to select any further buttons / actions.
I agree. Sorry if I was not clear, I actually meant to delegate the decision to you by saying that "we didn't find the right compromise yet", but I wasn't explicit with that.
Assignee | ||
Comment 5•2 years ago
|
||
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Comment 7•2 years ago
|
||
Backed out for causing bc failures in browser/components/urlbar/tests/browser-tips/browser_selection.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/f80a792ac5d811335bfabf50281e1cddf32dcda3
Assignee | ||
Updated•2 years ago
|
Comment 9•2 years ago
|
||
bugherder |
Comment 10•2 years ago
|
||
Since nightly and release are affected, beta will likely be affected too.
For more information, please visit auto_nag documentation.
Assignee | ||
Updated•2 years ago
|
Comment 11•2 years ago
|
||
Confirming this issue as verified fixed on 112.0a1(20230223094032) and 111.0b4(20230221190142) using Windows 11, macOS 13 and Ubuntu22.
Description
•