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

macOS system UI appearance within Brave should match Brave theme color #1289

Closed
cndouglas opened this issue Sep 26, 2018 · 4 comments · Fixed by brave/brave-core#1805
Closed
Assignees
Labels
OS/macOS priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-macOS QA/Yes release-notes/include

Comments

@cndouglas
Copy link
Contributor

This applies to macOS 10.14 (Mojave) or later only.

Even if the Brave UI is set to the dark theme, system UI (context menus, dialog boxes, etc.) does not match the dark theme.


The system UI appearance should match the Brave theme color setting:

Note that this is the opposite of #1189. That issue is for Brave's theme to take the appearance of the system. This issue is for the system UI within Brave to take Brave's theme appearance.

To do this, set the NSAppearance of the shared NSApplication. Something like this:

if (theme == light) {
    NSApplication.sharedApplication.appearance = [NSAppearance appearanceNamed:NSAppearanceNameAqua];
} else if (theme == dark) {
    NSApplication.sharedApplication.appearance = [NSAppearance appearanceNamed:NSAppearanceNameDarkAqua];
}
@bbondy bbondy added this to the 1.x Backlog milestone Sep 30, 2018
@rebron rebron added the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label Oct 5, 2018
@simonhong simonhong self-assigned this Nov 13, 2018
@rebron rebron modified the milestone: 1.x Backlog Feb 7, 2019
@simonhong
Copy link
Member

@bbondy @petemill Should we increase this priority also?

@rebron rebron added priority/P4 Planned work. We expect to get to it "soon". priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P5 Not scheduled. Don't anticipate work on this any time soon. priority/P4 Planned work. We expect to get to it "soon". labels Mar 1, 2019
@cndouglas
Copy link
Contributor Author

This is partially done. As of 0.62.24 dev, the system UI now matches the macOS theme appearance. This is a great improvement if your macOS and Brave theme settings match. In dark mode (macOS and Brave):


When brave/brave-core#1805 is implemented, the system UI will match the Brave theme appearance. This will improve the experience for users whose macOS and Brave theme settings do not match.

@LaurenWags
Copy link
Member

@kjozwiak since you have Mojave could you test this one when you get the chance?

@kjozwiak
Copy link
Member

Verification PASSED on macOS 10.14.3 x64 using the following build:

Brave 0.63.14 Chromium: 73.0.3683.75 (Official Build) dev(64-bit)
Revision 909ee014fcea6828f9a610e6716145bc0b3ebf4a-refs/branch-heads/3683@{#803}
OS Mac OS X
  • Used both Brave Colors: Dark & Brave Colors: Same as MacOS

Examples:

Screen Shot 2019-03-19 at 11 40 59 PM

Screen Shot 2019-03-19 at 11 41 55 PM

Screen Shot 2019-03-19 at 11 44 20 PM

Screen Shot 2019-03-19 at 11 43 36 PM

Screen Shot 2019-03-19 at 11 44 51 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS/macOS priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-macOS QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants