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
fix: dark mode tray opacity on Mojave #19013
Conversation
6f2b9db
to
d3ea248
Compare
@codebytere I assume you're not running catalina, I'd like to verify this on a catalina machine before we declare it fixed. I can test this afternoon if no one else has a catalina machine available |
d3ea248
to
d051a58
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested this and it isn't quite working afaics, pulling down and building locally
bc2dcfb
to
1037c3d
Compare
1037c3d
to
50c7885
Compare
Release Notes Persisted
|
I have automatically backported this PR to "5-0-x", please check out #19082 |
Description of Change
Closes #19009
Closes #19006.
Fixes an issue which is fixed in
master
as a result of this Chromium CL.Previously Chromium forward-declared
NSAppearanceNameDarkAqua
name as anNSString*
, so our updated functions in #18666 would never believe themselves to be in dark mode.This fixes that issue by removing Chromium's older forward declarations and adding them in ourselves.
cc @MarshallOfSound
Checklist
npm test
passesRelease Notes
Notes: Fixed an issue where the Tray icon would never believe itself to be in dark mode.