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

New tab pages shows blank settings propagated to all windows types including PT and TOR #12456

Closed
GeetaSarvadnya opened this issue Nov 2, 2020 · 3 comments · Fixed by brave/brave-core#7021

Comments

@GeetaSarvadnya
Copy link

Description

New tab pages shows blank settings propagated to all windows types including PT and TOR

Steps to Reproduce

  1. Clean profile 1.18.x
  2. Open brave://settings
  3. Change New tab pages shows settings to Blank page
  4. Open an NTP in a normal window ( NTP is blank as expected)
  5. Open a PT window and TOR tab window

Actual result:

image
image

Expected result:

New tab pages shows blank settings should be applicable for Normal window types only as PT and TOR windows types have the custom background for NTP's

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.18.25 Chromium: 87.0.4280.40 (Official Build) nightly (64-bit)
Revision 349b9adbfb2784db684617c7731a57d1e61b3be0-refs/branch-heads/4280@{#863}
OS Windows 10 OS Version 1903 (Build 18362.1139)

Version/Channel Information:

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

Other Additional Information:

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

Miscellaneous Information:

cc: @brave/legacy_qa @bsclifton @rebron @karenkliu

@karenkliu
Copy link

Correct, the blank new tab page option should only be for normal windows.

@rebron
Copy link
Collaborator

rebron commented Nov 2, 2020

cc: @simonhong Can you take a look?

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Nov 2, 2020
@simonhong simonhong self-assigned this Nov 2, 2020
@kjozwiak kjozwiak added this to the 1.18.x - Nightly milestone Nov 4, 2020
@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Nov 19, 2020

Verification passed on


Brave | 1.18.57 Chromium: 87.0.4280.66 (Official Build) beta (64-bit)
-- | --
Revision | fd98a29dd59b36f71e4741332c9ad5bda42094bf-refs/branch-heads/4280@{#1432}
OS | Windows 10 OS Version 2004 (Build 19041.572)

  • Verified STR from the description and ensured blank new Tab page settings is applicable only for Normal window tabs

Normal window tabs:
image

Private window tabs
image

TOR window tabs
image

Guest window tabs
image


Verification passed on

Brave 1.18.62 Chromium: 87.0.4280.67 (Official Build) dev (64-bit)
Revision 0e5d92df40086cf0050c00f87b11da1b14580930-refs/branch-heads/4280@{#1441}
OS Ubuntu 18.04 LTS
  • Verified STR from the description and ensured blank new Tab page settings is applicable only for Normal window tabs

Normal window tabs:
image

Private window tabs
image

TOR window tabs
image

Guest window tabs
image

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.

6 participants