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

welcome screen: support dark mode #3911

Closed
rebron opened this issue Mar 27, 2019 · 3 comments · Fixed by brave/brave-core#2909
Closed

welcome screen: support dark mode #3911

rebron opened this issue Mar 27, 2019 · 3 comments · Fixed by brave/brave-core#2909
Assignees
Labels
about-pages/welcome design A design change, especially one which needs input from the design team priority/P5 Not scheduled. Don't anticipate work on this any time soon. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/exclude

Comments

@rebron
Copy link
Collaborator

rebron commented Mar 27, 2019

Welcome screens should support dark theme when set by OS preference.

Description

Placeholder for dark theme screenshots of Welcome Tour.

@rebron rebron added the design A design change, especially one which needs input from the design team label Mar 27, 2019
@rebron rebron added the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label Mar 29, 2019
@rossmoody
Copy link
Contributor

Welcome Light
Welcome Dark

@imptrx
Copy link

imptrx commented Jul 25, 2019

Test Plan:

  1. Ensure OS is dark mode enabled; ensure theme is set to System theme (default) on welcome or Same as ____OS in settings page
  2. Change system OS, welcome page should change accordingly
  3. All welcome screens should be dark mode enabled

@btlechowski
Copy link

btlechowski commented Sep 20, 2019

Verification passed on

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Ubuntu 18.04 LTS

Verification passed on

Brave 0.69.128 Chromium: 77.0.3865.75 (Official Build) (64-bit)
Revision 201e747d032611c5f2785cae06e894cf85be7f8a-refs/branch-heads/3865@{#776}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Verification PASSED on macOS 10.14.6 x64 using the following build:

Brave 0.69.129 Chromium: 77.0.3865.90 (Official Build) (64-bit)
Revision 58c425ba843df2918d9d4b409331972646c393dd-refs/branch-heads/3865@{#830}
OS macOS Version 10.14.6 (Build 18G95)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
about-pages/welcome design A design change, especially one which needs input from the design team priority/P5 Not scheduled. Don't anticipate work on this any time soon. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants