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

Does not switch between dark/light modes without restart #1091

Closed
stackptr opened this issue Sep 28, 2019 · 2 comments
Closed

Does not switch between dark/light modes without restart #1091

stackptr opened this issue Sep 28, 2019 · 2 comments

Comments

@stackptr
Copy link
Contributor

stackptr commented Sep 28, 2019

It's unclear to me if this is as intended (i.e., not implemented) or if I am experiencing a bug.

Reading this guide on implementing dark mode in Electron, it seems like the check on the current "mode" is via api.systemPreferences.isDarkMode which I see in source/browser.ts: setDarkMode(). Yet that guide also mentions the AppleInterfaceThemeChangedNotification which I cannot grep for.

I can work on adding this if it is indeed missing but I wanted to check here first

Version: 2.38.0 (2.38.0.1617)

@stackptr
Copy link
Contributor Author

The issue I'm experiencing around the "Follow system appearance" option is referenced but not elaborated on in these other issues: #1010 #1046

@iandrewt
Copy link

iandrewt commented Oct 9, 2019

Supposedly this requires electron 7.0.0 to fix properly (at least in Catalina), or an Info.plist file with NSRequiresAquaSystemAppearance.

Electron 7 isn't out yet (it's in beta), so we might need to wait for that before this can be fixed. According to Electron's release timeline, that should be October 22nd.

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

No branches or pull requests

2 participants