Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Clarify search suggestions toggle and expand on-device functionality beyond Google #43762

Closed
3 of 6 tasks
ShivanKaul opened this issue Feb 5, 2025 · 0 comments · Fixed by brave/brave-core#27526
Closed
3 of 6 tasks
Assignees
Labels
feature/search feature/settings needs-text-change This change requires some careful wording. OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include

Comments

@ShivanKaul
Copy link
Collaborator

ShivanKaul commented Feb 5, 2025

Description

If Show search suggestions is turned off in brave://settings/search, we still use an on-device model to generate suggestions based on what the user is typing in the omnibox. However, we only do this for Google. Also, the current text is not super clear.

PROPOSAL:

  1. Use on-device model no matter what the default search engine is (not just Google)
  2. Change toggle label and description to be:

Toggle name: Improve search suggestions
Toggle description: When on, Brave sends what you type in the address bar or search box to your default search engine to get better suggestions. When off, Brave only shows suggestions generated on your device. This is always off in private windows.

Steps to reproduce

  1. Go to brave://settings/search
  2. Check Show search suggestions toggle and description
  3. Turn OFF
  4. See if you get search suggestions using local model (i.e. no new network requests to any search engine)
  5. Try with a few different default search engines (including Google)

Actual result

  1. Currently the toggle text is not completely accurate (we still "show" search suggestions if the toggle is off using the local model when Google is the default search provider, and the description is correct but not complete)
  2. We only use the local model if default search engine is Google

Expected result

See PROPOSAL above.

Reproduces how often

Easily reproduced

Brave version (brave://version info)

1.77.5 Chromium: 133.0.6943.39 (Official Build) nightly (arm64)

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

  • with Brave Shields disabled
  • with Brave Rewards disabled
  • in the latest version of Chrome

Miscellaneous information

No response

@ShivanKaul ShivanKaul added feature/search feature/settings needs-text-change This change requires some careful wording. OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include labels Feb 5, 2025
@rebron rebron moved this to In progress in Settings Feb 6, 2025
@brave-builds brave-builds added this to the 1.77.x - Nightly milestone Feb 10, 2025
@rebron rebron moved this from In progress to Completed in Settings Feb 18, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
feature/search feature/settings needs-text-change This change requires some careful wording. OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

3 participants