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)
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•11 months ago
|
Comment 1•11 months 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•10 months ago
|
||
It may be a good idea to coordinate with UX on the best solution here.
Updated•10 months ago
|
Assignee | ||
Updated•9 months ago
|
Comment 3•9 months 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•9 months 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•9 months ago
|
||
Updated•9 months ago
|
Updated•9 months ago
|
Updated•9 months ago
|
Updated•9 months ago
|
Assignee | ||
Updated•9 months ago
|
Assignee | ||
Updated•9 months ago
|
Updated•9 months ago
|
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/357170b2f52a Fix down/up arrow behavior with multiple selectable elements per row. r=adw
Comment 7•8 months 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
Pushed by dgottwald@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2ef3c1d1f5d8 Fix down/up arrow behavior with multiple selectable elements per row. r=adw
Assignee | ||
Updated•8 months ago
|
Comment 9•8 months ago
|
||
bugherder |
Comment 10•8 months ago
|
||
Since nightly and release are affected, beta will likely be affected too.
For more information, please visit auto_nag documentation.
Assignee | ||
Updated•8 months ago
|
Comment 11•7 months ago
|
||
Confirming this issue as verified fixed on 112.0a1(20230223094032) and 111.0b4(20230221190142) using Windows 11, macOS 13 and Ubuntu22.
Description
•