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

Fixes brave://version Brave logo in dark mode. #4149

Merged
merged 1 commit into from
Dec 6, 2019

Conversation

mkarolin
Copy link
Collaborator

@mkarolin mkarolin commented Dec 5, 2019

Fixes brave/brave-browser#7221

Copied product_logo.png to product_logo_white.png to be used in dark
mode.

Submitter Checklist:

Test Plan:

Reviewer Checklist:

  • New files have MPL-2.0 license header.
  • Request a security/privacy review as needed.
  • Adequate test coverage exists to prevent regressions
  • Verify test plan is specified in PR before merging to source

After-merge Checklist:

  • The associated issue milestone is set to the smallest version that the
    changes has landed on.
  • All relevant documentation has been updated.

@mkarolin mkarolin added this to the 1.4.x - Nightly milestone Dec 5, 2019
@mkarolin mkarolin self-assigned this Dec 5, 2019
Fixes brave/brave-browser#7221

Copied product_logo.png to product_logo_white.png to be used in dark
mode.

Chromium change:

https://chromium.googlesource.com/chromium/src/+/7ccedcb1069f7a3ae1a17f835bdfa148bdad326a

commit 7ccedcb1069f7a3ae1a17f835bdfa148bdad326a
Author: Robbie Gibson <rkgibson@google.com>
Date:   Thu Oct 10 10:20:24 2019 +0000

    Add dark mode styling to chrome://version

    The uses the <picture> element to select different sources for the logo
    image based on the device's mode. It also moves the white logo files
    to the same place as the existing files for consistency. There are
    corresponding CLs to move the google_chrome logo files as well.

    Bug: 1006216
@mkarolin mkarolin force-pushed the maxk-fix-version-logo branch from 27760e4 to fa96922 Compare December 5, 2019 19:28
@bsclifton
Copy link
Member

CI looks good (had a failure on macOS install?)

Pulling down the code now to verify fix 😄

Copy link
Member

@bsclifton bsclifton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Beautiful! 👍
Screen Shot 2019-12-05 at 10 47 35 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

brave://version page shows Chromium logo/name on dark theme
2 participants