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

Allow user to show/hide background images on New Tab Page #4523

Closed
karenkliu opened this issue May 21, 2019 · 10 comments · Fixed by brave/brave-core#2668
Closed

Allow user to show/hide background images on New Tab Page #4523

karenkliu opened this issue May 21, 2019 · 10 comments · Fixed by brave/brave-core#2668

Comments

@karenkliu
Copy link

karenkliu commented May 21, 2019

Test plan

See brave/brave-core#2668 (comment)

Description

Provide users with the ability to show/hide background images on the New Tab page.

Designs

Add a new icon on the NTP that represents Dashboard settings:

r0 background images 1

Clicking or tab selecting the icon shows a menu with dashboard settings:

r0 background images 2

Clicking or tab selecting the toggle keeps the menu open while turning off the background image:

r0 background images 4

Clicking anywhere outside the menu closes it:

r0 background images 5

Design Specs

r0 background images specs

The Dashboard Settings menu should support dark theme set at the OS level:
r0 background images dark theme

Abstract link for more CSS/layout: https://share.goabstract.com/9563ae1d-93af-40ef-811b-4c3f384fe8f4

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label May 21, 2019
@karenkliu
Copy link
Author

@imptrx reach out to @rossmoody for guidance on front-end components in our design system

@srirambv
Copy link
Contributor

@karenkliu with no background image is it going to be the gradient colour? or is that customized to pick the system colour scheme because Windows and Mac now supports dark theme to be used?

@karenkliu
Copy link
Author

@srirambv to start the no background image will be a gradient color. We will of course be supporting more options for the background image in the future (pick your own image, set to a solid color, pick a solid color from color picker, pick a different gradient) etc.

@srirambv
Copy link
Contributor

Huge +1 for custom colour picker for new tab page.

@petemill
Copy link
Member

@karenkliu There are now 4 buttons here that look the same, but do very different things - one unintrusively pops up a toggle, the others load whole different pages. Can we make it intuitive that 3 of them are links to different pages, and 1 of them will lead to a menu? Perhaps by putting an arrow on that button.

@karenkliu
Copy link
Author

@petemill We can temporarily resolve that by making new icons for Settings, Bookmarks, and History that has an arrow on them.
Or, we can wait for planned updates to the Top Sites behavior, where the user can add/remove any site, including browser pages such as Settings, Bookmarks, and History, to that section. At this point we would remove the persistent icons at the bottom right, leaving just the Dashboard Settings icon. Need to discuss with @rebron whether we want to invest in the stopgap solution (new icon) or be OK with this discrepancy for now while waiting for fixes to the Top Sites widget.

@rebron
Copy link
Collaborator

rebron commented Jun 12, 2019

@karenkliu Stopgap with new icons per @petemill's suggestion.

@karenkliu
Copy link
Author

@rebron I created a separate issue #4930 to update the icons.

@bsclifton
Copy link
Member

Updated labelling to help with QA and edited original post to link to test plan 👍

@bsclifton bsclifton changed the title show/hide background images on New Tab Page Allow user to show/hide background images on New Tab Page Jun 25, 2019
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jul 21, 2019

Verification passed on

Brave 0.68.106 Chromium: 76.0.3809.62 (Official Build) dev (64-bit)
Revision 7b77856b3aa34d72f246d12340fc1ded8b2c0e83-refs/branch-heads/3809@{#798}
OS Windows 10 OS Version 1803 (Build 17134.523)

image

  • Verified New tab page by changing the Theme and Brave color

Verification passed on

Brave 0.68.109 Chromium: 76.0.3809.62 (Official Build) dev (64-bit)
Revision 7b77856b3aa34d72f246d12340fc1ded8b2c0e83-refs/branch-heads/3809@{#798}
OS Ubuntu 18.04 LTS

Verified passed with

Brave 0.68.116 Chromium: 76.0.3809.87 (Official Build) beta (64-bit)
Revision 111fe1e15d5ced26080a7dc239bcfe70f6c49aad-refs/branch-heads/3809@{#967}
OS Mac OS X

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