Closed Bug 1480508 Opened Last year Closed Last year
Add Top Site Search Shortcuts automatically for new and existing users
52 bytes, text/x-github-pull-request
|Details | Review|
We would like to automatically add a Top Site Search Shortcut to users' for existing users if they have an open (i.e. unpinned) tile available and the host name (google, amazon) is found in their top sites. Note that we should store some kind of state in persistent storage when we do this so that we do not add a second time if it is removed/edited/replaced.
Assignee: nobody → ahillier
Priority: P2 → P1
Summary: Add Top Site Search Shortcuts for automatically existing users who visit those sites → Add Top Site Search Shortcuts automatically for existing users who visit those sites
The logic for converting the available search engines to special @aliased search will need to be duplicated from bug 1480504. See https://phabricator.services.mozilla.com/D2681 where it does Services.search.getEngines() then checking _isDefault and a global Map of prefix name matching "Google" and "Amazon"
Later on, this might be cleaned up by having activity stream import ChromeUtils.import("resource://gre/modules/PlacesSearchAutocompleteProvider.jsm") to get the same Map ?
(See bug 1480509 comment 3) We will only add the @google search shortcut if the user doesn't have Google as their default search engine.
Summary: Add Top Site Search Shortcuts automatically for existing users who visit those sites → Add Top Site Search Shortcuts automatically for new and existing users
Update 8/6/18: Google/Amazon should be pinned in that order to the first open position in everyone's Top Sites (regardless of frecency).
Commit pushed to master at https://github.com/mozilla/activity-stream https://github.com/mozilla/activity-stream/commit/725532f6f8e922944ab28c484f38a1c933770ab7 Fix Bug 1480508 - add topSiteSearchShortcuts for new and existing users (#4289)
I have verified this on Windows 10 x64, Arch Linux and Mac 10.13.3 with the latest version of "Firefox Nightly" and I can confirm that the "Top Search Shortcuts" replace the existing top site cards of the same type (if those cards are unpinned) for older profiles, and if for new profiles, the "Top Site Search Shortcuts" are added automatically.
I have verified that the search shortcuts override preexisting Top Sites of the same type and that they are automatically added for new profiles in the latest Beta (62.0b18 Build ID 20180816151750) on Windows 10, Mac 10.13 and Arch Linux x64.
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.