Closed Bug 1962200 Opened 9 months ago Closed 3 months ago

Pressing Option + Up / Down should open unified search button

Categories

(Firefox :: Address Bar, enhancement, P2)

enhancement

Tracking

()

VERIFIED FIXED
146 Branch
Tracking Status
firefox146 --- verified

People

(Reporter: daleharvey, Assigned: daleharvey)

References

(Blocks 1 open bug)

Details

(Whiteboard: [sng-scotchbonnet-followon][scotchbonnet-unifiedsearchbutton])

Attachments

(1 file)

No description provided.
Priority: -- → P2
Whiteboard: [sng-scotchbonnet] → [sng-scotchbonnet-followon]
Assignee: nobody → daisuke
Status: NEW → ASSIGNED

Hi Dale!
I tried to implement this one, but it might be impossible to do on the current element we are using.

My approach was to add the key event listener to the component and scroll when we detect the event.
We changed the popup component from <panel> to <menupopup> in bug 1934410 though, it seems that we can't detect key events for the menupopup element. Also, looks like there is no way to change the scroll behavior for inside of menupopup.
Thus, I don't think we can do it by my approach, but do you have another approach?

Flags: needinfo?(dharvey)
Whiteboard: [sng-scotchbonnet-followon] → [sng-scotchbonnet-followon][scotchbonnet-unifiedsearchbutton]
Severity: -- → N/A
Duplicate of this bug: 1947144

Same bug, (slightly) broader scope :
Like the current search menu works, pressing the Up and Down arrow keys should seamlessly navigate between search suggestions and the list of search engines
(As not every keyboard user is a power user, keyboard navigation of the search menu should not require previous knowledge an obscure shortcut like Option + Up/Down)

Blocks: 1974218

(In reply to gui97cham from comment #5)

Same bug, (slightly) broader scope :
Like the current search menu works, pressing the Up and Down arrow keys should seamlessly navigate between search suggestions and the list of search engines
(As not every keyboard user is a power user, keyboard navigation of the search menu should not require previous knowledge an obscure shortcut like Option + Up/Down)

I agree, this is a very important aspect. πŸ‘ Just plain up/down would also return the keyboard shortcut feature to how it was before 138, thus making it easier for users.

Hey Daisuke

Apologies for taking so long to get to this, so I think if we want the wrap around behaviour here we probably need to talk to someone more familiar with the popup / menulist components.

However I am not sure the wrap around behaviour is the most important part, I think one of the main benefits here is to open the unified search popup while the urlbar is focused without having to alt+tab to the actual button, once the popup is open then up / down can be used to scroll the engines and the OPTION is ignored. I think that code is somewhere around https://searchfox.org/mozilla-central/source/browser/components/urlbar/UrlbarController.sys.mjs#304

I think that along is worth fixing, dropped the ball here so happy to pick it up if unless you want to go again? Cheers

Flags: needinfo?(dharvey)

(In reply to Dale Harvey (:daleharvey) from comment #7)

I think that along is worth fixing, dropped the ball here so happy to pick it up if unless you want to go again? Cheers

Hey Daisuke, are you going to pick this up again?

Flags: needinfo?(daisuke)
Summary: Pressing Option + Up / Down should scroll engines / open unified search button → Pressing Option + Up / Down should open unified search button

Hi DΓ£o, thank you very much for your ping!

Dale, if it is possible to do this after fixing realtime suggestions, I want to do this.
If this is urgent, could you take this one, unfortunately?

Flags: needinfo?(daisuke) → needinfo?(dharvey)

(In reply to Daisuke Akatsuka (:daisuke) from comment #9)

If this is urgent, could you take this one, unfortunately?

From my perspective, it would be nice to have this in preparation for the new search bar implementation where we intend to also bring in the unified search button instead of one-off search buttons.

No worries I will give it a shot

Assignee: daisuke → dharvey
Flags: needinfo?(dharvey)
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 146 Branch
QA Whiteboard: [search] [qa-triage-done-c146/b145] [qa-ver-needed-c146/b145]
QA Contact: dlucaci

Reproduced with Firefox 145.0a1 (2025-10-29) on macOS 15.
Verified as fixed with Firefox 146.0a1 (20251028094950) on macOS 15, Win 10 and Ubuntu 22.

Status: RESOLVED → VERIFIED
QA Whiteboard: [search] [qa-triage-done-c146/b145] [qa-ver-needed-c146/b145] → [search] [qa-triage-done-c146/b145] [qa-ver-done-c146/b145]
Blocks: 2007267
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: