-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
feat: add darwinDarkModeSupport option to support mojave dark mode for older Electron versions #893
Conversation
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.
I think it should be darwinDarkModeSupport
? There aren't any other options that are prefixed by enable
.
docs/api.md
Outdated
*Boolean* | ||
|
||
Forces support for Mojave dark mode in your packaged app, this sets the `NSRequiresAquaSystemAppearance` | ||
key to `false` in your apps `Info.plist`. For more information see the |
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.
app's
docs/api.md
Outdated
|
||
Forces support for Mojave dark mode in your packaged app, this sets the `NSRequiresAquaSystemAppearance` | ||
key to `false` in your apps `Info.plist`. For more information see the | ||
[apple developer documentation](https://developer.apple.com/documentation/appkit/nsappearancecustomization/choosing_a_specific_appearance_for_your_app) |
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.
Apple
Period at the end
c4324c2
to
1468aa6
Compare
Codecov Report
@@ Coverage Diff @@
## master #893 +/- ##
==========================================
- Coverage 94.48% 94.35% -0.14%
==========================================
Files 12 12
Lines 617 620 +3
==========================================
+ Hits 583 585 +2
- Misses 34 35 +1
Continue to review full report at Codecov.
|
…ode for older electron versions
1468aa6
to
d57ed03
Compare
Also, this should be in |
The AppVeyor failure is #892 |
I don't know why codecov asserts that your PR loses coverage. |
Refs: https://developer.apple.com/documentation/appkit/nsappearancecustomization/choosing_a_specific_appearance_for_your_app