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

Ability to modify New Tab Page Sponsored Images from Settings #8760

Closed
jonathansampson opened this issue Mar 19, 2020 · 3 comments · Fixed by brave/brave-core#5151
Closed

Comments

@jonathansampson
Copy link
Contributor

jonathansampson commented Mar 19, 2020

Description

Users who are attempting to modify/disable New Tab Page Sponsored Images are at times looking within about:settings (Brave's global Settings) and failing to locate controls for NTP SIs. Presently, this feature is controlled from the New Tab Page itself, by clicking the Dashboard button at the bottom-right.

Brave version (brave://version info)

1.5.113

Additional Information

Sponsored Images are enabled and disabled from within general Settings on iOS and Android.

Designs

Accessed via brave://settings/newtab

Add a "New Tab Page" section in Settings.

new tab page

The controls shown in Settings are duplicates linked to the controls on the new tab page:

Screen Shot 2020-03-20 at 3 43 29 PM

Assets

icon asset.zip

@mandar-brave
Copy link

@rebron please prioritize. Please make settings a dupe of what is available on NTP.
cc @bradleyrichter

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Mar 20, 2020
@karenkliu
Copy link

Designs added!

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented May 5, 2020

Verification passed on


Brave | 1.9.55 Chromium: 81.0.4044.129 (Official Build) dev (64-bit)
-- | --
Revision | 3d71af9f5704a40b85806f4d08925db24605ba25-refs/branch-heads/4044@{#979}
OS | Windows 10 OS Version 1803 (Build 17134.1006)

Verification passed on

Brave 1.9.61 Chromium: 81.0.4044.138 (Official Build) dev (64-bit)
Revision 8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{# 999}
OS Linux
  • Verified disabling Background images disables Sponsored images as well in both settings and dashboard
    image
  • Verified that Toggling an option in New Tab Settings reflected on the NTP dashboard settings and vice versa.
  • Verified that settings has a new section for NTP settings as per design
    image

Verified passed with

Brave	1.9.62 Chromium: 81.0.4044.138 (Official Build) dev (64-bit)
Revision	8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999}
OS	macOS Version 10.14.6 (Build 18G3020)

Screen Shot 2020-05-11 at 9 31 45 AM

Screen Shot 2020-05-11 at 9 32 15 AM

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