Open Bug 1940591 Opened 1 month ago Updated 5 days ago

Typing w does not prompt to search for wikipedia

Categories

(Firefox :: Address Bar, defect, P2)

defect

Tracking

()

People

(Reporter: daleharvey, Assigned: daleharvey)

References

Details

(Whiteboard: [sng-scotchbonnet])

Attachments

(1 file)

Typing w will not match wikipedia, typing "e" will (if ebay is not enabled). This is because the code that matches engines was copied from the current autofill implementation that attempts to match the domain of the engine and not the engine name. The previous implementation had to ensure the muxer could deduplicate the hueristic result.

We do not need to worry about that with actions as the autofill result / hueristic are seperate from the actions and will not be considered duplicated either way. The match should match simply with the engine name for consistency / simplicitys sake.

Assignee: nobody → dharvey

(In reply to Dale Harvey (:daleharvey) from comment #0)

Typing w will not match wikipedia, typing "e" will (if ebay is not enabled). This is because the code that matches engines was copied from the current autofill implementation that attempts to match the domain of the engine and not the engine name.

In reality the code tries to also match subdomains, so "en.wiki" would match "w", that's how the old tab to search did. Either there's a bug in the copied code, or wikipedia doesn't go over the autofill threshold.
The reason to go through autofill was to check that we're not suggesting an engine that is barely or even never used. What is the replacement for that heuristic? Or are we ok suggesting any installed engine even if it's not commonly used? Like if I never used ebay, and I type e, do we expect to suggest searching on ebay?

Flags: needinfo?(dharvey)

on my profile typing w suggests searching on wikipedia

I assume the previous tabtosearch implementation had to worry about not suggesting too much as these were urlbar results that would affect muscle memory however with this implementation I think that is less of a consideration. If a user is only typing “e” then going straight to the result then will be doing that on muscle memory (and providing the action wont change anything), if they match more than that it seems very likely they want the engine.

Josh has mentioning thinking we are currently quite conservative about showing actions so we can check in with him but I think we are likely ok with showing rarely used engines here. Obviously we will want to keep track but I think there is a justification for a simpler hueristic here.

Flags: needinfo?(dharvey)

(In reply to Dale Harvey (:daleharvey) from comment #4)

I assume the previous tabtosearch implementation had to worry about not suggesting too much as these were urlbar results that would affect muscle memory

The concern was more about not suggesting things the user doesn't care about. e.g. if I don't use ebay, why should I search on it? But typing e or eb I'd be constantly suggested to do that.
I'm ok with a simpler heuristic, maybe just check the domain was "recently" visited, we could use the revHost field to check for any subdomain. We could cache the info for the session so we don't hit disk on every search. Or maybe there's something even easier (store engine last used datetime?).
I agree with consulting Josh, I may just be overzealous.

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: