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)
Firefox
Search
Tracking
()
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.
Assignee | ||
Comment 1•7 years ago
|
||
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]
Assignee | ||
Comment 2•7 years ago
|
||
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
Updated•7 years ago
|
Flags: qe-verify?
Priority: -- → P2
Updated•7 years ago
|
Flags: qe-verify? → qe-verify+
QA Contact: gwimberly
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 56.4 - Aug 1
Priority: P2 → P1
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
mozreview-review |
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
Comment 6•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Updated•7 years ago
|
status-firefox54:
--- → unaffected
status-firefox55:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Verified on Windows, Mac, and Ubuntu.
Updated•7 years ago
|
Flags: qe-verify+
Updated•7 years ago
|
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•