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

Dark title bar on macOS + matching sidebar #221

Closed
ylluminate opened this issue Dec 20, 2019 · 11 comments
Closed

Dark title bar on macOS + matching sidebar #221

ylluminate opened this issue Dec 20, 2019 · 11 comments

Comments

@ylluminate
Copy link

Other macOS Electron apps have had an issue with having a light app title bar like ElectronMail is having presently:
image

Other apps such as Mattermost have now resolved this through Electron updates:
mattermost/desktop#848

In addition to this, it would be nice to change the left bar's color scheme to match the dark/light mode of the OS. Perhaps leaving it the way it is by default for when the OS is in light mode, but then when in dark mode, it would be nice to change the left side panel to a darker gray and black mode to match the OS.

@vladimiry
Copy link
Owner

@ylluminate, the first point should be easy to enable if @electron got the feature supported. The only issue is that I won't be able to test the feature in the near future so will ask you to do so.

The second point will cause adding custom macOS related code since at the moment detecting the dark/light mode toggling in @electron is only supported for macOS. I would prefer to avoid adding such code but have not decided yet. Although I see that https://electronjs.org/docs/api/native-theme stuff got supported, it worth looking into it as it's supported for all systems since depends on chromium mechanisms.

@ylluminate
Copy link
Author

ylluminate commented Dec 20, 2019

Be happy to test. Also - honestly - even in light mode, making the left side bar dark by default wouldn't hurt aesthetics at all and would be better to default to dark for such a drawer / sidebar mechanism since it's mostly out of the way and does not need focus even in light mode.

@ylluminarious I'm sure you'd be happy to test as well.

@vladimiry
Copy link
Owner

Currently used left bar's color was chosen as a compromise between Protonmail's dark-ish colors and Tuatnota's light colors as the app supported both providers. But Tutanota support gt dropped with v4.0.0 and so I agree that it won't hurt to darken the color of the left bar.

@vladimiry
Copy link
Owner

https://electronjs.org/docs/api/native-theme support is added since @electron v7. The app currently uses v6 and the goal is to switch to v8 bypassing the v7. The switch is currently blocked by several upstream / @electron issues.

@ylluminate
Copy link
Author

😢 So so sad. 😞

@vladimiry
Copy link
Owner

Besides we will probably need to switch to Protonmail v4 to enable dark mode support in the app (like at least left bar's color toggling) since v3 doesn't seem to be very darky.

@vladimiry
Copy link
Owner

Meanwhile, if you hate the left/top bar color so much and you use only one account you could try hiding that bar by hitting F12 or via the general app settings, see #128 (the feature enabled since v3.6.0).

vladimiry added a commit that referenced this issue Dec 20, 2019
* ref #221
* won't be working yet as @electron "native-theme" thing is not enabled so far
@vladimiry
Copy link
Owner

vladimiry commented Dec 20, 2019

@ylluminate can you try the dark title bar thing enabled with pre-v4 release?

FYI: this version also removes Tutanota accounts from the app settings:

vladimiry added a commit that referenced this issue Dec 20, 2019
* ref #221
* won't be working yet as @electron "native-theme" thing is not enabled so far
@ylluminate
Copy link
Author

Excellent, dark titlebar is good and I'm detecting no problems @vladimiry. IDEA: you could allow left sidebar / drawer customization by users like the notification colors. This way we could simply plug in our own colors to our pleasure and save yourself from implementation of mode switching logic.

@vladimiry
Copy link
Owner

@ylluminate please indicate if the issue got resolved in 4.2.0.

@ylluminate
Copy link
Author

@vladimiry this is looking much better, thank you! 🚀

Dropdown still white on the gear down arrow, but you're doing great here. Ultimately ProtonMail itself needs to get full on dark themes working since it's still pretty clunky.

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