Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Incorrect setting toggle for NTP images #2083

Closed
srirambv opened this issue Dec 5, 2019 · 3 comments
Closed

Incorrect setting toggle for NTP images #2083

srirambv opened this issue Dec 5, 2019 · 3 comments

Comments

@srirambv
Copy link
Contributor

srirambv commented Dec 5, 2019

Description:

Incorrect setting toggle for NTP images

Steps to Reproduce

https://bravesoftware.slack.com/archives/C06UXF3KJ/p1575551486274100

Actual result:
https://bravesoftware.slack.com/archives/C06UXF3KJ/p1575551502274300?thread_ts=1575551486.274100&cid=C06UXF3KJ

Expected result:
https://bravesoftware.slack.com/archives/C06UXF3KJ/p1575551512275100?thread_ts=1575551486.274100&cid=C06UXF3KJ

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easy

Brave Version:
1.14.1(19.12.05.00)

Device details:
All

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@mandar-brave
Copy link

@jhreis @anthonypkeane this is a requirement for NTP. Spec reference here
New section under Brave Settings>(Display)New Tab Page with a new page containing settings: “Show Background Images” default ON, and an exposed setting for “Show Sponsored Images” also ON, when enabled.

@anthonypkeane
Copy link

anthonypkeane commented Dec 7, 2019 via email

@btlechowski
Copy link

btlechowski commented Jan 28, 2020

Verification passed on 1.14.3(20.01.28.18) on Production on iPhone SE (iOS 12.4)

  • Verified that Disabling the Background images in Settings also disables Sponsored Images
  • Verified that Enabling the Background images in Settings also enables Sponsored Images
  • Verified that Sponsored images can't be enabled when Background images are disabled

Verification passed on 1.14.3 (20.01.28.18) on Production on iPhone 11 (iOS 13.3)

  • Verified that Disabling the Background images in Settings also disables Sponsored Images
  • Verified that Enabling the Background images in Settings also enables Sponsored Images
  • Verified that Sponsored images can't be enabled when Background images are disabled
  • Verified that Sponsored images can be disabled when Background images are enabled

Verification passed on 1.14.3(20.01.28.18) on Production on iPad 5th Gen (iOS 12)

  • Verified that Disabling the Background images in Settings also disables Sponsored Images
  • Verified that Enabling the Background images in Settings also enables Sponsored Images
  • Verified that Sponsored images can't be enabled when Background images are disabled

Verification passed on 1.14.3(20.01.28.18) on Production on iPad Mini (iOS 13)

  • Verified that Disabling the Background images in Settings also disables Sponsored Images
  • Verified that Enabling the Background images in Settings also enables Sponsored Images
  • Verified that Sponsored images can't be enabled when Background images are disabled

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