Skip to content
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

relabel search engine name to "Brave" #16564

Closed
rebron opened this issue Jun 23, 2021 · 5 comments · Fixed by brave/brave-core#9216
Closed

relabel search engine name to "Brave" #16564

rebron opened this issue Jun 23, 2021 · 5 comments · Fixed by brave/brave-core#9216

Comments

@rebron
Copy link
Collaborator

rebron commented Jun 23, 2021

Description

We currently have search engine name set as Brave Search beta but the label looks awkward when searching from the url bar.

Search engine name should just be Brave and searches from url bar should show [query] - Brave Search

Steps to Reproduce

  1. Download 1.26.67.
  2. Change default search engine to Brave Search beta from brave://settings/search
  3. Perform a search and notice [query] - Brave Search beta Search

Actual result:

image__42_

Expected result:

When searching from URL bar -
Screen Shot 2021-06-22 at 6 17 31 PM

Manage Settings drop down -
Screen Shot 2021-06-22 at 6 21 05 PM

Other Search engines list -
Screen Shot 2021-06-22 at 6 22 00 PM

Reproduces how often:

Easily.

Brave version (brave://version info)

1.26.67

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Does the issue resolve itself when disabling Brave Rewards? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Miscellaneous Information:

@rebron rebron changed the title relabel search engine name to Brave relabel search engine name to "Brave" Jun 23, 2021
@bsclifton bsclifton self-assigned this Jun 23, 2021
@rebron rebron added priority/P3 The next thing for us to work on. It'll ride the trains. release-notes/exclude labels Jun 23, 2021
bsclifton added a commit to brave/brave-core that referenced this issue Jun 23, 2021
@bsclifton bsclifton added this to the 1.28.x - Nightly milestone Jun 23, 2021
@stephendonner
Copy link

stephendonner commented Jun 24, 2021

Verified PASSED using

Brave 1.26.73 Chromium: 91.0.4472.124 (Official Build) (x86_64)
Revision 7345a6d1bfcaff81162a957e9b7d52649fe2ac38-refs/branch-heads/4472_114@{#6}
OS macOS Version 11.4 (Build 20F71)

Went through two scenarios re: upgrading from 1.26.67 --> 1.26.73 Chromium: 91.0.4472.124.

  • Scenario #1: Brave Search isn't set as the default
  • Scenario #2: Brave Search is set as the default

Ensured that the correct text/labels were being used for the two above scenarios. Also ensured that previously selected search engines stay as default after upgrading.

  • Scenario #1: Brave Search isn't set as the default and upgraded to 1.26.73 Chromium: 91.0.4472.124
Example Example Example Example Example Example
Screen Shot 2021-06-29 at 9 22 21 AM Screen Shot 2021-06-29 at 9 23 02 AM Screen Shot 2021-06-29 at 9 27 42 AM Screen Shot 2021-06-29 at 9 29 14 AM Screen Shot 2021-06-29 at 9 34 54 AM Screen Shot 2021-06-29 at 9 29 52 AM
  • Scenario #2: Brave Search is set as the default and upgraded to 1.26.73 Chromium: 91.0.4472.124
Example Example Example Example Example Example Example
Screen Shot 2021-06-29 at 9 32 57 AM Screen Shot 2021-06-29 at 9 33 17 AM Screen Shot 2021-06-29 at 9 33 25 AM Screen Shot 2021-06-29 at 9 34 48 AM Screen Shot 2021-06-29 at 9 34 51 AM Screen Shot 2021-06-29 at 9 34 54 AM Screen Shot 2021-06-29 at 9 35 06 AM

Verification PASSED on Win 10 x64 using the following build:

Brave | 1.28.34 Chromium: 91.0.4472.124 (Official Build) nightly (64-bit)
--- | ---
Revision | 7345a6d1bfcaff81162a957e9b7d52649fe2ac38-refs/branch-heads/4472_114@{#6}
OS | Windows 10 OS Version 2009 (Build 19042.1081)

Went through two scenarios re: upgrading from 1.28.23 Chromium: 91.0.4472.114 --> 1.28.34 Chromium: 91.0.4472.124.

  • Scenario #1: Brave Search isn't set as the default
  • Scenario #2: Brave Search is set as the default

Ensured that the correct text/labels were being used for the two above scenarios. Also ensured that previously selected search engines stay as default after upgrading.

  • Scenario #1: Brave Search isn't set as the default and upgraded to 1.28.34 Chromium: 91.0.4472.124
Example Example Example Example Example
notDefault1 notDefault2 notDefault3 notDefault4 notDefault5
  • Scenario #2: Brave Search is set as the default and upgraded to 1.28.34 Chromium: 91.0.4472.124
Example Example Example Example
default1 default2 default3 default4

Verification PASSED on Samsung S10+ running Android 11 using the following build:

Brave | 1.28.34 Chromium: 91.0.4472.124

Test Case #1 (Default Install)

Example Example Example
Screenshot_20210628-214057_Brave - Nightly Screenshot_20210628-214117_Brave - Nightly Screenshot_20210628-214123_Brave - Nightly

Test Case #2 (Update & Brave Search not default SE)

Upgraded from 1.28.22 Chromium: 91.0.4472.114 --> 1.28.34 Chromium: 91.0.4472.124 and ensured that Brave was being used instead of Brave Search beta. Also ensured that the previous default SE hasn't been changed.

Example Example
Screenshot_20210628-215819_Brave - Nightly Screenshot_20210628-215824_Brave - Nightly

Test Case #3 (Update & Brave Search is set as default SE)

Upgraded from 1.28.22 Chromium: 91.0.4472.114 --> 1.28.34 Chromium: 91.0.4472.124 and ensured that Brave was being used instead of Brave Search beta.

Example Example Example
Screenshot_20210628-220042_Brave - Nightly Screenshot_20210628-221532_Brave - Nightly Screenshot_20210628-221537_Brave - Nightly

@bsclifton bsclifton added the OS/Android Fixes related to Android browser functionality label Jun 28, 2021
@kjozwiak
Copy link
Member

@brave/legacy_qa note that seeing Brave Search beta being displayed after upgrading on Android is expected as per:

Screenshot_20210628-220042_Brave - Nightly

@kjozwiak
Copy link
Member

Clearing QA Pass-macOS so we can make sure the above is working correctly once we get 1.26.x RC's as Brave Search is pretty important 👍

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jun 29, 2021

Verification passed on


Brave | 1.26.73 Chromium: 91.0.4472.124 (Official Build) (64-bit)
-- | --
Revision | 7345a6d1bfcaff81162a957e9b7d52649fe2ac38-refs/branch-heads/4472_114@{#6}
OS | Windows 10 OS Version 2004 (Build 19041.1052)


Went through two scenarios re: upgrading from 1.26.67 --> 1.26.73 Chromium: 91.0.4472.124.

  • Scenario #1: Brave Search isn't set as the default
  • Scenario #2: Brave Search is set as the default

Ensured that the correct text/labels were being used for the two above scenarios. Also ensured that previously selected search engines stay as default after upgrading.

  • Scenario #1: Brave Search isn't set as the default and upgraded to 1.26.73 Chromium: 91.0.4472.124
1.26.67 1.26.67 1.26.67 1.26.73 1.26.73 1.26.73
image image image image image image
  • Scenario #2: Brave Search is set as the default and upgraded to 1.26.73 Chromium: 91.0.4472.124
1.26.67 1.26.73 1.26.73 1.26.73 1.26.73 1.26.73
image image image image image image

Verified passed with

Brave	1.26.73 Chromium: 91.0.4472.124 (Official Build) (64-bit)
Revision	7345a6d1bfcaff81162a957e9b7d52649fe2ac38-refs/branch-heads/4472_114@{#6}
OS	Linux
Scenario 1: Clean install of 1.26.73

Confirmed expected text/labels displayed in below areas.

Example Example Example Example Example Example Example Example
clean1 clean2 clean3 clean4 clean5 clean6 clean7 clean8
Scenario 2: Upgrade from 1.26.67 (Brave Search is not default SE)

Confirmed expected text/labels displayed in below areas. Confirmed previously set default SE remained default SE. Screenshots from 1.26.67 for comparison.

1.26.67 1.26.73
1 26 67 1 73 upgrade google default 1
1 26 67 2 73 upgrade google default 2
1 26 67 3 73 upgrade google default 3
1 26 67 4 73 upgrade google default 4
1 26 67 5 73 upgrade google default 5

Also confirmed in this case that if you change default SE to Brave after upgrading, expected text shows in URL bar:

73 upgrade google default 6
Scenario 3: Upgrade from 1.26.67 (Brave Search is default SE)

Confirmed expected text/labels displayed in below areas. Confirmed previously set default SE remained default SE. Screenshots from 1.26.67 for comparison.

1.26.67 1.26.73
1 26 67 brave default 1 73 upgrade brave default 1
1 26 67 brave default 2 73 upgrade brave default 2
1 26 67 brave default 3 73 upgrade brave default 3
1 26 67 brave default 4 73 upgrade brave default 4
1 26 67 brave default 5 73 upgrade brave default 5

@srirambv
Copy link
Contributor

Verification passed on OnePlus 6T with Android 10 running 1.26.74 x64 build
Clean Install

  • Verified Brave Search beta is updated to Brave during onboarding
  • Verified selecting Brave during onboarding sets it as default for both normal and private tabs
image image image image

**Upgrade Install (with Brave as default)

  • Verified upgrading to 1.26.74 updates the Search engine name for both standard and private tabs in selection view
  • Verified upgrading to 1.26.74 retains Brave Search beta on Search engines overview page
image image

Upgrade Install (with Brave not set as default)

  • Verified upgrading retains the SE list correctly and default SE
  • Verified upon upgrade Brave Search beta is replaced by Brave for both standard and private tabs in selection view

Verification passed on Samsung Tab A with Android 10 running 1.26.74 x64 build
Clean Install

  • Verified Brave Search beta is updated to Brave during onboarding
  • Verified selecting Brave during onboarding sets it as default for both normal and private tabs
image image image image

**Upgrade Install (with Brave as default)

  • Verified upgrading to 1.26.74 updates the Search engine name for both standard and private tabs in selection view
  • Verified upgrading to 1.26.74 retains Brave Search beta on Search engines overview page
image image

Upgrade Install (with Brave not set as default)

  • Verified upgrading retains the SE list correctly and default SE
  • Verified upon upgrade Brave Search beta is replaced by Brave for both standard and private tabs in selection view

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants