1657212 - Support key-modified one-offs and their context menus in update 2

Status: VERIFIED FIXED

(Firefox :: Address Bar, task, P1)

Description

[Marco Bonardo [:mak]]

Today we asked UX what to do about key modifiers, so for example when a user CTRL/SHIFT+clicks a one-off button.
The current proposal is that, if there is a search string, it will act as the old buttons, directly executing a query in a new tab/window.
If there isn't a search string, it may open a new tab/window with the urlbar focused and search mode active on that engine. Though, I suppose we could initially just make it ignore the modifier and split this empty string part to a separate bug, if the patch starts to complicate.

Comment 1

[Harry Twyford]

I imagine we should use the same behaviour for the "Search in New Tab" one-off context menu item.

Comment 3

[Harry Twyford]

This bug is closely tied to what we do with one-off keyboard shortcuts going forward. We identified some inconsistencies in the current shortcuts and this is a good opportunity to fix them. This is also blocked on what we decide to do with the one-off context menu, so I'm marking this blocked-ux and will pause work on it.

Comment 4

[Harry Twyford]

We're going to support Shift+Click and Shift+Enter executing a search immediately in the current tab (i.e. how one-offs behaved before update2). All other keyboard shortcuts will behave as they do now. For example, Alt+Enter will continue to search immediately in a new foreground tab and Accel+Click will continue to search in a new background tab. Shift+Enter previously opened a one-off search in a new window, so we're dropping support for that behaviour. The user can still open a one-off search in a new window by selecting a one-off with the keyboard and then Shift+Clicking on the heuristic result or on the Go button.

We're also going to remove the context menu entirely from the one-offs. Both items in the context menu can be achieved by other means. Also, the context menu doesn't make any sense for the local one-offs and it is confusing that the engine one-offs have a context menu but the local one-offs do not.

Comment 5

[Harry Twyford]

Attachment: Bug 1657212 - Support one-off key modifiers and remove one-off context menus. r?adw!

This patch adds support for key modifiers on the refreshed one-offs. They work the same as the old key modifiers, with one exception: Shift+Click and Shift+Enter both execute a search immediately in the current tab, replicating the behaviour of the old one-offs. For empty searches, Shift+Click and Shift+Enter just enter search mode. To support these Shift modifiers, we dropped support for opening a one-off search in a new window. Users can still search in a new window with a non-default engine by typing a search string, selecting a one-off with the keyboard, then Shift+Clicking the heuristic result or the Go button.

Other key modifiers worth pointing out include Accel+Click to search in a new background tab and Alt+Enter to search in a new foreground tab. If these modifiers are used on the local one offs or with an empty query, we open search mode in the new tab.

This needs tests, but I'm posting this tentative version to get feedback on the design.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bd0fa7757307
Support one-off key modifiers and remove one-off context menus. r=adw

Comment 7

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/bd0fa7757307

Comment 8

[Cristian Comorasu, QA [:ccomorasu], Releae Desktop QA - Please contact [e-mail]]

I verified this issue using 82.0a1 (2020-09-14) on macOS 10.13 and Windows 10 x64.
Adrian could you help me with Ubuntu verification?

Comment 9

[Petruta Horea [:phorea], Desktop QA]

Verified on latest Nightly 83.0a1 under Ubuntu 18.04 64-bit.