Skip to content
This repository was archived by the owner on May 12, 2026. It is now read-only.

enable dark mode support for macOS mojave#2509

Merged
Rokt33r merged 4 commits into
BoostIO:masterfrom
lvarado:mojave-dark-mode
Apr 28, 2019
Merged

enable dark mode support for macOS mojave#2509
Rokt33r merged 4 commits into
BoostIO:masterfrom
lvarado:mojave-dark-mode

Conversation

@lvarado

@lvarado lvarado commented Oct 19, 2018

Copy link
Copy Markdown
Contributor

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

lvarado commented Oct 19, 2018

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown

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

@Rokt33r

Rokt33r commented Feb 1, 2019

Copy link
Copy Markdown
Member

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

@ericsolomon

Copy link
Copy Markdown

Any update on this?

@lvarado

lvarado commented Apr 22, 2019

Copy link
Copy Markdown
Contributor Author

@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

lvarado commented Apr 22, 2019

Copy link
Copy Markdown
Contributor Author

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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

5 participants