Show only autosuggest results when search only engine alias is typed in the address bar

VERIFIED FIXED in Firefox 64

Status

()

P1
normal
VERIFIED FIXED
6 months ago
a month ago

People

(Reporter: mikedeboer, Assigned: adw)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox64+ verified)

Details

(Whiteboard: [fxsearch], URL)

Attachments

(1 attachment)

(Reporter)

Description

6 months ago
For a visual explanation, please see Verdi's second bulleted mockup over at https://mozilla.invisionapp.com/share/4POCDI07U9A#/screens/322966230_Seach_Bar_Changes_For_63

If possible, we'd try and make sure that whenever a search engine alias is entered followed by keywords, to show only the autosuggest results in the AwesomeBar.
(Assignee)

Updated

6 months ago
Assignee: nobody → adw
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [fxsearch]
(Assignee)

Comment 1

6 months ago
[Tracking Requested - why for this release]:

Moving the tracking? flag from bug 1496772 to this bug, per bug 1496772 comment 5:

(In reply to Mike de Boer [:mikedeboer] from bug 1496772 comment #0)
> [Tracking Requested - why for this release]:
> 
> There are a number of - should be - low effort changes to improve the search
> tiles/ keywords user interaction.
> We believe this will greatly impact the general usefulness, usability and
> users' perception of the feature when it launches in Firefox 63.
tracking-firefox64: --- → ?
tracking-firefox64: ? → +
(Assignee)

Comment 2

5 months ago
This is based on the patch in bug 1496811.

This patch looks a little worse than it probably is. (Maybe not by much.) Some of this is indentation changes, moving code around, renaming, and adding jsdocs. It looks like we missed the boat on uplifting this (and the other couple of bugs) to 63, so there's not a super-pressing need to keep the patch minimal.

PlacesSearchAutocompleteProvider assumes you're fetching suggestions from the current engine, so I had to modify it to take an engine. While I was doing that, I got a little frustrated with some of its implementation, naming, and interface. It seems like it was written to be a little more generic than it ended up being? There doesn't seem to be any need for it to return generic "match" objects instead of simply engines and `{ suggestion, historical }` objects, for example. The "defaultMatch" concept also doesn't make much sense IMO, especially with the aforementioned changes. So I made some improvements, hopefully, and I also added some jsdocs.

Comment 4

5 months ago
Pushed by dwillcoxon@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/665fecc38740
Show only search suggestion results when only an engine alias and query are typed in the address bar r=mak

Comment 5

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/665fecc38740
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
status-firefox64: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64

Comment 6

5 months ago
Using the arrow keys to highlight a search suggestion ends up removing the search shortcut from the urlbar. Is this intentional? To be more specific:

STR:
1. Type `@google test`
2. Use down arrow key to highlight say the second suggestion

AR: urlbar says `testbook`. If I continue typing at this point, the search query is no longer being sent to Google but to my default engine.

ER: urlbar says `@google testbook` and Google remains "active." This feels more intuitive to me and allows[1] for further editing of the search query.

Also after STR1, the one-off section says `Search for @google test with:` followed by the usual list. After STR2, it says `Search for testbook with:`. Bug 1498023 will fix this, iiuc.


[1]Unsure if I'm explaining it correctly so as an example: I would like to be able to do the following:

Goal: Search Google for `telescope price`. Google is not the default engine.

Expected "lazy" steps:
i) Type `@google teles`
ii) Use down arrow to pick the `telescope` suggestion. At this point, urlbar should read `@google telescope` and continue to be in "queries being sent to Google" mode
iii) Type out ` p`. First suggestion is `telescope price`. Select it, urlbar changes to `@google telescope price`, press enter.

Updated

5 months ago
Duplicate of this bug: 1294387
(Assignee)

Comment 8

5 months ago
Yeah, these are good points, Bruce.  I realized that the @engine text went away when you arrow down to the suggestions, but I didn't think about the fact that changing the suggestion and then selecting it would search with the default engine.  A lot of these interactions are underspecified tbh.  I'll file a new bug and mark it blocking this one.

Thanks for pointing out all the little corners where this stuff could use improvement, it's been helpful. :-)
(Assignee)

Updated

5 months ago
Depends on: 1500516
This is verified fixed using Firefox 64.0b3 (BuildId:20181022150107) on the following OSes: Windows 10 x64, Ubuntu 18.04 x64 and macOS 10.13.
Status: RESOLVED → VERIFIED
status-firefox64: fixed → verified
Flags: qe-verify+
(Assignee)

Updated

a month ago
Depends on: 1529642
You need to log in before you can comment on or make changes to this bug.