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

enable dark mode support for macOS mojave #2509

Merged
merged 4 commits into from Apr 28, 2019

Conversation

lvarado
Copy link
Contributor

@lvarado lvarado commented Oct 19, 2018

Dark mode support for macOS Mojave was added to electron-packager in 12.2.0:
electron/packager#893

This PR updates electron-packager and adds the necessary config to the gruntfile.

Before (dark mode enabled):
screen shot 2018-10-19 at 12 11 29 pm

After:
screen shot 2018-10-19 at 12 12 22 pm

Dark mode is automatically used when it is enabled in the system configuration:
screen recording 2018-10-19 at 12 22 21 pm

Fixes #2417

@lvarado
Copy link
Contributor Author

lvarado commented Oct 19, 2018

Note: you need to build the app with grunt pre-build to test. Dark mode will not be enabled if you simply run it with yarn dev because electron-packager is not used in that case.

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Oct 20, 2018
@ZeroX-DG ZeroX-DG requested a review from Rokt33r October 20, 2018 16:47
@neha-sri
Copy link

Just a little note for @lvarado - Thank you for this! Waiting for this to be merged and released.

@Rokt33r
Copy link
Member

Rokt33r commented Feb 1, 2019

Awesome! Could you fix the conflicts? I'll merge this soon.

@ericsolomon
Copy link

Any update on this?

@lvarado
Copy link
Contributor Author

lvarado commented Apr 22, 2019

@Rokt33r @ericsolomon got the conflicts resolved and cleaned up yarn.lock so it only includes hashes present in master for a cleaner diff. Should be all set now.

@lvarado
Copy link
Contributor Author

lvarado commented Apr 22, 2019

Also related: #1055 and #2318

@Rokt33r Rokt33r removed the awaiting review ❇️ Pull request is awaiting a review. label Apr 28, 2019
@Rokt33r Rokt33r added this to the v0.11.15 milestone Apr 28, 2019
@Rokt33r Rokt33r merged commit 2999e49 into BoostIO:master Apr 28, 2019
@lvarado lvarado deleted the mojave-dark-mode branch April 30, 2019 18:58
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.

[Feature Request] Support dark mode of MacOS Mojave
5 participants