-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add new setting to show/hide bookmarks and history from the address bar autocomplete results #22948
Comments
Edited description to include more spec |
If the idea is to be privacy-friendly by default, shouldn't the initial setting for the "Browsing history" preference be false? That would be more in line with the general sentiment in #12292, IIUC. |
@waldyrious I worry that it would degrade the default user experience a bit, as all of a sudden a bunch of things that used to show up would stop. I know that the omnibox is the way I normally find things (i.e. I remember, vaguely, a bit of the title of the page I visitied and it shows up in my suggestions). I wonder if it would be worth having a separate toggle for search results @petemill? It looks like this is exposed and wouldn't be too hard to add. It does seem like it might be hard to explain the difference though, so I dunno. |
In addition, I wonder if these settings, rather than living the appearance or privacy section or settings, should live in the search settings, as that seems what they're most closely related too. It seems like these new settings should be grouped with |
Test Plan
Note: When you disable 'Browsing history' you also don't get tab suggestions (i.e. searching for an open tab). This is expected behavior, though not ideal. |
Added to 1.42.x milestone 😄 |
Verification
|
Brave | 1.42.65 Chromium: 103.0.5060.114 (Official Build) beta (x86_64) |
---|---|
Revision | a1c2360c5b02a6d4d6ab33796ad8a268a6128226-refs/branch-heads/5060@{#1124} |
OS | macOS Version 13.0 (Build 22A5295i) |
Steps:
- installed
1.42.65
- launched Brave
- opened
brave://settings/appearance
- compared to
1.41.96
- confirmed
Show autocomplete suggestions in address bar
was checked - confirmed
Bookmarks
&Browsing History
were added - confirmed all options except for
Suggested Sites
wereEnabled
1.42.65 |
1.41.96 |
---|---|
- toggled
Show autocomplete suggestions in address bar
toDisabled
and confirmed it collapsed the subcategories. Toggled it back toEnabled
. - opened a new tab and searched for
Hello
- clicked on the first result and bookmarked it
- typed
Hel
into the search bar. Confirmed search suggestion forHello
and a suggestion for the YouTube page I bookmarked. - disabled the
Browsing History
toggle and typedhel
into the search bar. Confirmed suggestion for the bookmarked YouTube page but not for 'hello' from my search history. - disabled the 'Bookmarks' toggle and type
hel
into the search bar. Confirmed option to search forhel
but no suggestions forhello
.
step 1 | step 2 & 3 | step 4 | step 5 | step 5(b) | step 6 | step 6(b) |
---|---|---|---|---|---|---|
Migrations/upgrades
From 1.41.96
->
1.42.65
:
- default prefs -> default prefs
- launched
1.41.96
- opened
brave://settings/appearance
- shut down Brave
- renamed
Brave-Browser
profile toBrave-Browser-Beta
- launched
1.42.65
- confirmed:
Show autocomplete suggestions in address bar
is toggled toEnabled
Top sites
is toggled toEnabled
Browsing History
is toggled toEnabled
Bookmarks
is toggled toEnabled
1.41.96 |
1.42.65 |
---|---|
- all prefs disabled -> all prefs disabled
- launched
1.41.96
- opened
brave://settings/appearance
- toggled
Show top sites in autocomplete suggestions
toDisabled
- toggled
Show top sites in autocomplete suggestions
toDisabled
- renamed
Brave-Browser
profile toBrave-Browser-Beta
- launch
1.42.65
- confirm
Show autocomplete suggestions in address bar
is toggled toDisabled
(which collapses the rest of the settings)
1.41.96 |
1.42.65 |
---|---|
- Disabled
Show autocomplete in address bar
- launched
1.40.113
- opened
brave://settings/appearance
- toggled
Show autocomplete in address bar
toDisabled
- shut down Brave
- renamed
Brave-Browser
profile toBrave-Browser-Beta
- launched
1.42.65
- confirmed
Show autocomplete suggestions in address bar
is toggled toDisabled
(which collapses the rest of the settings)
1.41.96 |
1.42.65 |
---|---|
From 1.40.113
-> 1.42.65
:
- default prefs -> default prefs
- launched
1.40.113
- opened
brave://settings/appearance
- shut down Brave
- renamed
Brave-Browser
profile toBrave-Browser-Beta
- launched
1.42.65
- confirmed:
Show autocomplete suggestions in address bar
is toggled toEnabled
Top sites
is toggled toEnabled
Browsing History
is toggled toEnabled
Bookmarks
is toggled toEnabled
1.40.113 |
1.42.65 |
---|---|
- all prefs disabled -> all prefs disabled
- launched
1.40.113
- opened
brave://settings/appearance
- toggled
Show top sites in autocomplete suggestions
toDisabled
- toggled
Show top sites in autocomplete suggestions
toDisabled
- renamed
Brave-Browser
profile toBrave-Browser-Beta
- launch
1.42.65
- confirm
Show autocomplete suggestions in address bar
is toggled toDisabled
(which collapses the rest of the settings)
1.40.113 |
1.42.65 |
---|---|
- Disabled
Show top sites in autocomplete suggestions
Verification PASSED on
Clean profileCase 1: Verify default autocomplete suggestion switches behavior_PASSED
Case 1: Top sites in autocomplete will match against Alexa 500 sites
Case 2: Autocomplete suggestion from seach history
Case 3: Autocomplete suggestion from the bookmarks
Case 2: Turn ON all the autocomplete suggestion switches_PASSED
Case 3: Turn OFF all the subcategory autocomplete suggestion switches_PASSED
Case 4: Turn OFF induvidual autocomplete suggestion switches_PASSEDCase 1: Turn OFF
Case 2: Turn OFF
Case 3: Turn OFF
Case 5: Turn OFF show autocomplete suggestion main switch_PASSED
Upgade profileCase 1: Upgrade without making any changes to autocomplete settings_PASSED
Case 2: Turn OFF `Show top sites autocomplete` settings in 1.41 and then upgrade to 1.42_PASSED
Case 3: Enable `Show suggested sites in autocomplete suggestions` settings in 1.41 and then upgrade to 1.42_PASSED
Verification passed on
Clean profileCase 1: Verify default autocomplete suggestion switches behavior_PASSED
Case 1: Top sites in autocomplete will match against Alexa 500 sites
Case 2: Autocomplete suggestion from seach history
Case 3: Autocomplete suggestion from the bookmarks
Case 2: Turn ON all the autocomplete suggestion switches_PASSED
Case 3: Turn OFF all the subcategory autocomplete suggestion switches_PASSED
Case 4: Turn OFF induvidual autocomplete suggestion switches_PASSEDCase 1: Turn OFF
Case 2: Turn OFF
Case 3: Turn OFF
Case 5: Turn OFF show autocomplete suggestion main switch_PASSED
Upgade profileCase 1: Upgrade without making any changes to autocomplete settings_PASSED
Case 2: Turn OFF `Show top sites autocomplete` settings in 1.41 and then upgrade to 1.42_PASSED
Case 3: Enable `Show suggested sites in autocomplete suggestions` settings in 1.41 and then upgrade to 1.42_PASSED
|
For others coming across this: in Brave 1.48, this option can be found under the "Appearance" heading in Settings. It's called "Show autocomplete suggestions in address bar". If the setting is enabled, check-boxes for more fine-grained control is shown. |
Description
Users should have the ability to show or hide results from bookmarks and history in the address bar (omnibox).
These settings should live in:
brave://settings/appearance
(also considering these settings to live in brave://settings/privacy)
Design
Expected result:
Brave version (brave://version info)
Target release 1.41.x
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
#12292
Test Plan
#22948 (comment)
The text was updated successfully, but these errors were encountered: