59 bytes, text/x-review-board-request
I used to have the search box in the hamburger menu. I was able to use Ctrl-K to pop open the menu, enter my search in the search box, hit enter, and have the search open in a new tab. At some point the search box disappeared from the hamburger menu, and Ctrl-K now focuses on the address bar. I can re-add the search box to the overflow panel using the menu customization tool. However, Ctrl-K does not open the overflow panel.
This is effectively a regression, so I think we just need to make sure we fix it given people do this, especially given the plans to remove the search bar from the default nav-bar configuration.
Whiteboard: [photon-structure][triage] → [photon-structure]
bug 1354117 covers moving added items from the hamburger panel into the overflow panel (and vice versa if you flip the pref). I'll morph this bug to explicitly be about the shortcut.
Depends on: 1354117
Summary: Search box disappears from hamburger menu post-photon → accel/ctrl/cmd-k should bring up the search box if it's in the overflow panel post-photon
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 56.4 - Aug 1
Priority: P2 → P1
Comment on attachment 8890849 [details] Bug 1378807 - make search bar shortcut work in the permanent overflow panel, https://reviewboard.mozilla.org/r/162074/#review167498 Looks good to me. Tested on Windows with the searchbar pinned to the panel, overflowed to the panel, not anywhere, in the toolbar, and even in the tabstrip! Thanks! :D
Attachment #8890849 - Flags: review?(bwinton) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/87bd87bb4064 make search bar shortcut work in the permanent overflow panel, r=bwinton
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox56: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
status-firefox54: --- → unaffected
status-firefox55: --- → unaffected
status-firefox-esr52: --- → unaffected
Verified on Windows, Mac, and Ubuntu.
Status: RESOLVED → VERIFIED
status-firefox57: --- → verified
You need to log in before you can comment on or make changes to this bug.