Closed Bug 1664760 Opened 2 months ago Closed 1 month ago

Order of results in search mode is wrong if "Show search suggestions before history" is unchecked

Categories

(Firefox :: Address Bar, defect, P2)

defect
Points:
2

Tracking

()

RESOLVED FIXED
83 Branch
Iteration:
83.1 - Sept 21 - Oct 4
Tracking Status
firefox83 --- fixed

People

(Reporter: mak, Assigned: mak)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

When restricting to search (with ?) we used to put suggestions first, history after, basically we use matchBucketsSearch instead of matchBuckets.
This is done here: https://searchfox.org/mozilla-central/source/browser/components/urlbar/UrlbarMuxerUnifiedComplete.jsm#129
Apparently the heuristic doesn't have anymore RESULT_TYPE.SEARCH in search mode, and that breaks the ordering.

I'm probably going to change the RESULT_TYPE in bug 1663686, that may also fix this bug.

Priority: P3 → P2
Assignee: nobody → mak
Status: NEW → ASSIGNED
Iteration: --- → 82.2 - Sep 7 - Sep 20
Points: --- → 2
Summary: Order of results in search mode looks off → Order of results in search mode is wrong if "Show search suggestions before history" is unchecked

Ok, I was misremebering, we stopped setting matchBucketsSearch, and that means there's no special sorting depending on the heuristic result... Though it still seems to make sense to put history later in search mode with an engine.

In reality we are now discussing again whether browsing history should be shown or not. So I guess I won't fix this in bug 1663686 for now.

No longer depends on: 1663686

from a long discussion in #search, while here I should look at why restyleSearches is removing all the non-restyled results, make the redirects filter in UnifiedComplete depending on that pref (filter targets if we restyle).

  • always keep search suggestions for
    • use restyleSearches only in search mode to get cleaner history results and dedupe against search history
    • filter redirects differently depending on restyleSearches
Iteration: 82.2 - Sep 7 - Sep 20 → 83.1 - Sept 21 - Oct 4
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/35a5c311e239
Improve the set of results when in search mode for a non-general search engine. r=adw
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in before you can comment on or make changes to this bug.