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

Refresh Private Window New Tab page #18486

Closed
rebron opened this issue Sep 30, 2021 · 8 comments · Fixed by brave/brave-core#10413
Closed

Refresh Private Window New Tab page #18486

rebron opened this issue Sep 30, 2021 · 8 comments · Fixed by brave/brave-core#10413

Comments

@rebron
Copy link
Collaborator

rebron commented Sep 30, 2021

Description

Update and simplify private window new tab page. Show DDG toggle for users who have previously set DDG as their search engine. New users and existing users who have never set the toggle should not be shown the DDG setting.

New Design

All users:

image

Existing users with DDG on:

image

Current Design for Reference

Screen Shot 2021-09-30 at 2 33 41 PM

Brave version (brave://version info)

n/a

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? n/a
  • Can you reproduce this issue with the nightly channel? n/a

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:

Assets

Figma: https://www.figma.com/file/5THkuEtO2Ewn9LfqrHZP9a/?node-id=544%3A401

@rebron rebron added OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. feature/search QA/Yes release-notes/include labels Sep 30, 2021
@rebron rebron assigned emerick and karenkliu and unassigned emerick and karenkliu Sep 30, 2021
@rebron rebron added priority/P2 A bad problem. We might uplift this to the next planned release. and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Oct 1, 2021
@emerick emerick added this to the 1.32.x - Nightly milestone Oct 8, 2021
@stephendonner
Copy link

Verified PASSED using

Brave 1.32.63 Chromium: 95.0.4638.40 (Official Build) nightly (x86_64)
Revision e3e7c76ba0284b16087cf4cf3153abfaef6470c7-refs/branch-heads/4638@{#624}
OS macOS Version 11.6 (Build 20G165)

Steps:

  1. Clean profile
  2. Set brave.use_alternate_private_search_engine preference to false
  3. Launched Brave
  4. Opened File | New Private Window
  5. Verified that Private Tab shows new design and no DuckDuckGo toggle
  6. Clean profile
  7. Set brave.use_alternate_private_search_engine preference to true (I set DuckDuckGo as search provider in 1.30.87, then used the profile in nightly)
  8. Launched Brave
  9. Opened File | New Private Window
  10. Verified that Private Tab shows new design and DuckDuckGo toggle
  11. Verified that toggle works as expected

Confirmed that when Search with DuckDuckGo is toggled to On, searches use DDG for search.

Confirmed that when Search with DuckDuckGo is toggled to Off, searches uses default browser.

example example example example example
Screenshot 2021-10-11 at 15 43 47 Screenshot 2021-10-11 at 16 26 50 Screenshot 2021-10-11 at 16 26 58 Screenshot 2021-10-11 at 16 34 31 Screenshot 2021-10-11 at 16 34 55

@nastavoi
Copy link

Hey, can you guys clarify that with these changes, user won't be able to set default search engine for private windows independently?

@rebron
Copy link
Collaborator Author

rebron commented Oct 17, 2021

@nastavoi The original implementation was only for DDG and it was a setting on the new tab page which wasn't necessarily a proper implementation in the sense that we'd ultimately like to offer more choices than a one choice toggle.

If an existing user had set DDG as their new tab page private window search engine, we'll display the DDG toggle. We want to honor that choice and don't want to make any changes there. New users from 1.31.x won't see that toggle though and we'll revisit how we set a search engine independently depending on window in brave://settings or we may consider a different implementation.

Hope that helps explain things.

@kjozwiak
Copy link
Member

Clearing QA Pass-macOS as we should check this again with the next 1.30.x RC.

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 18, 2021

Verification passed on


Brave | 1.31.86 Chromium: 95.0.4638.50 (Official Build) (64-bit)
-- | --
Revision | 3e9db37693d0cf8e31a89fbcf50401ae91125cef-refs/branch-heads/4638@{#816}
OS | Windows 10 Version 21H1 (Build 19043.1288)


Verified the test plan from brave/brave-core#10413

Clean profile
image

Upgrade profile

  • Install 1.30.89 DDG is OFF in Private Tab and upgrade the profile to 1.31.86
1.30.89 1.31.86
image image
  • Install 1.30.89 DDG is ON in Private Tab and upgrade the profile to 1.31.86
1.30.89 1.31.86
image image

Verification passed on

Brave 1.31.86 Chromium: 95.0.4638.50 (Official Build) (x86_64)
Revision 3e9db37693d0cf8e31a89fbcf50401ae91125cef-refs/branch-heads/4638@{#816}
OS macOS Version 11.6 (Build 20G165)

Verified the test plan from brave/brave-core#10413

Clean profile
Screen Shot 2021-10-18 at 12 49 33 PM

Upgrade profile

  • Install 1.30.89 DDG is OFF in Private Tab and upgrade the profile to 1.31.86
1.30.89 1.31.86
Screen Shot 2021-10-18 at 12 53 13 PM Screen Shot 2021-10-18 at 12 49 33 PM
  • Install 1.30.89 DDG is ON in Private Tab and upgrade the profile to 1.31.86
1.30.89 1.31.86
Screen Shot 2021-10-18 at 12 45 19 PM Screen Shot 2021-10-18 at 12 47 24 PM

Verification PASSED on PopOS 20.04 x64 using the following build:

Brave | 1.31.86 Chromium: 95.0.4638.50 (Official Build) (64-bit)
--- | ---
Revision | 3e9db37693d0cf8e31a89fbcf50401ae91125cef-refs/branch-heads/4638@{#816}
OS | Linux

Verified the STR/Cases mentioned in brave/brave-core#10413

Clean profile

image

Upgrade profile

  • Install 1.30.89 DDG is OFF in Private Tab and upgrade the profile to 1.31.86
1.30.89 1.31.86
image image
  • Install 1.30.89 DDG is ON in Private Tab and upgrade the profile to 1.31.86
1.30.89 1.31.86
image image

@DivyanshuAG
Copy link

I constantly shift between Google and DDG on the private window, but now that option is no longer visible. Is there a way to get it back?

@floriegl
Copy link

floriegl commented Nov 4, 2021

@DivyanshuAG just type ":d " before your search

@simonhong
Copy link
Member

@DivyanshuAG We are working on adding separte search engine setting for private window (brave/brave-core#13622)

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.

10 participants