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

brave://settings/themes does not work for SR/unable to turn off SR NTP images #9515

Closed
LaurenWags opened this issue Apr 28, 2020 · 2 comments

Comments

@LaurenWags
Copy link
Member

Description

Found while testing #8218

When installing using SR code, the brave://settings/themes page does not work. When you navigate there, nothing is selected (the SR radio button should be selected when installing from SR code). If you select an option and refresh the page, your selection is removed. If you select Brave default images and close the page, your selection is not respected so you can only view SR images on NTP.

Note - this seems to only be a problem in 1.8.x. Issue does not reproduce on 1.9.x or 1.10.x.

Steps to Reproduce

  1. Create empty profile dir
  2. Create promoCode text file in root profile dir and that file has TECHNIK string
  3. Load new tab and confirm you see Technikke image
  4. Open brave://settings/themes page
    --> no radio button selected (expected Technikke one to be selected)
  5. Select Brave default images radio button.
  6. Refresh page
    --> your selection is removed
  7. Select Brave default images again
  8. Open NTP.
    --> Technikke image displayed.

Actual result:

Screen Shot 2020-04-28 at 12 45 46 PM

Expected result:

Should be like this as it is in 1.9.x and 1.10.x
Screen Shot 2020-04-28 at 12 46 02 PM

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.8.85 Chromium: 81.0.4044.122 (Official Build) (64-bit)
Revision 44f4233f08910d83b146130c1938256a2e05b136-refs/branch-heads/4044@{#963}
OS macOS Version 10.14.6 (Build 18G3020)

Version/Channel Information:

  • Can you reproduce this issue with the current release? reproduces in 1.8.x
  • Can you reproduce this issue with the beta channel? no, does not reproduce in 1.9.x
  • Can you reproduce this issue with the dev channel? no, does not reproduce in 1.9.x
  • Can you reproduce this issue with the nightly channel? no, does not reproduce in 1.10.x

Other Additional Information:

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

Miscellaneous Information:

cc @rebron @bsclifton @brave/legacy_qa

@bsclifton
Copy link
Member

Fixed with brave/brave-core#5413

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented May 4, 2020

Verification passed on

Brave | 1.8.89 Chromium: 81.0.4044.129 (Official Build) (64-bit)
-- | --
Revision | 3d71af9f5704a40b85806f4d08925db24605ba25-refs/branch-heads/4044@{#979}
OS | Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified the STR from the description
  • ensured that the TECHNIK radio button is selected when Technikke image shown in NTP
    image
  • ensured that default brave images are shown in NTP when Brave default images radio button is selected
    image

Verification PASSED on macOS 10.15.4 x64 (Catalina) using the following build:

Brave | 1.8.89 Chromium: 81.0.4044.129 (Official Build) (64-bit)
-- | --
Revision | 3d71af9f5704a40b85806f4d08925db24605ba25-refs/branch-heads/4044@{#979}
OS | macOS Version 10.15.4 (Build 19E287)
  • verified the case outlined via brave://settings/themes does not work for SR/unable to turn off SR NTP images #9515 (comment)
  • ensured that the radio button stays visibly and has the correct item selected when switching between Brave default images and Technikke
  • ensured that refreshing brave://settings/themes doesn't reset the current selection
  • ensured that once Brave default images is selected, NTP displays the default images including the current NTP SI campaign
  • ensured that selecting Technikke correctly displays the correct images under NTP
  • ensured that the top tiles from Technikke are not restored once removed/edited

Screen Shot 2020-05-04 at 8 00 04 AM

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