Closed Bug 1378807 Opened 7 years ago Closed 7 years ago

accel/ctrl/cmd-k should bring up the search box if it's in the overflow panel post-photon

Categories

(Firefox :: Search, defect, P1)

defect

Tracking

()

VERIFIED FIXED
Firefox 56
Iteration:
56.4 - Aug 1
Tracking Status
firefox-esr52 --- unaffected
firefox54 --- unaffected
firefox55 --- unaffected
firefox56 --- fixed
firefox57 --- verified

People

(Reporter: catlee, Assigned: Gijs)

References

Details

(Keywords: regression, Whiteboard: [photon-structure])

Attachments

(1 file)

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.
Keywords: regression
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
Flags: qe-verify?
Priority: -- → P2
Flags: qe-verify? → qe-verify+
QA Contact: gwimberly
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 gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/87bd87bb4064
make search bar shortcut work in the permanent overflow panel, r=bwinton
https://hg.mozilla.org/mozilla-central/rev/87bd87bb4064
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Verified on Windows, Mac, and Ubuntu.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Blocks: 1387512
Flags: qe-verify+
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.