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

[Linux] dark mode switch follow gtk theme #74

Closed
icepie opened this issue Sep 3, 2022 · 15 comments
Closed

[Linux] dark mode switch follow gtk theme #74

icepie opened this issue Sep 3, 2022 · 15 comments

Comments

@icepie
Copy link

icepie commented Sep 3, 2022

Is your feature request related to a problem? Please describe.
I found a patch for implement the feat

Additional Notes
patch

@icepie icepie changed the title [Linux] dark mode switch with gtk theme [Linux] dark mode switch follow gtk theme Sep 3, 2022
@Alex313031
Copy link
Owner

@icepie Good idea! I will add this to Thorium, and credit you in the copyright header of the modified files

@Alex313031
Copy link
Owner

@icepie Here you go > and you can see i credited the original author and you > https://github.com/Alex313031/Thorium/blob/main/src/ui/gtk/native_theme_gtk.cc#L1

@icepie
Copy link
Author

icepie commented Sep 3, 2022

wow! thanks my bro!

@Alex313031
Copy link
Owner

@icepie Your welcome. Lol this is why i made the new issues templates, to encourage more things like this where people can actively improve thorium without having to actually modify code and push to the repo.

@Alex313031 Alex313031 reopened this Sep 6, 2022
@Alex313031
Copy link
Owner

@raisedintheusa
Copy link

raisedintheusa commented Sep 6, 2022

I'm not the biggest fan of this change. There's no way to disable this, right? There already was a flag to force dark mode if desired. I like a dark GTK theme for other applications with a more consistent design. And I like having the tab strip/address bar dark on Thorium. But websites are all so different that a global dark mode has a very inconsistent effect. Some sites are not made dark so I still require Dark Reader for them. Other sites become ugly or hard-to-read when made dark and there might be no option on the site to change it. Perhaps there could be a switch to disable this feature.

@Alex313031
Copy link
Owner

@raisedintheusa I can probably set it behind a chrome://flag and name it something like GTK Auto-Dark Mode
.

@Alex313031
Copy link
Owner

@icepie @raisedintheusa These patches should make a flag to enable/disable the feature. > 29e1d91 & d5fe7f8

@icepie
Copy link
Author

icepie commented Oct 9, 2022

@icepie @raisedintheusa These patches should make a flag to enable/disable the feature. > 29e1d91 & d5fe7f8

good work!

@Alex313031
Copy link
Owner

@icepie @raisedintheusa @tomByrer Latest release has this patch. It will be turned off by default (using the old behavior where one can manually control dark mode with the chrome://flags#force-dark-mode flag), but you can turn this flag on to allow auto gtk dark mode setting > chrome://flags#auto-dark-mode

@raisedintheusa
Copy link

raisedintheusa commented Oct 18, 2022

Latest release has this patch.

Thanks, works good!

@Alex313031
Copy link
Owner

@raisedintheusa @icepie Closing as completed.

@WingofaGriffin
Copy link

Is it possible to add these flags as sliders in the settings page? Mercury has the Firefox theme selector, and I think something similar in settings would be a nice feature for a more obvious dark mode.

@Alex313031
Copy link
Owner

@WingofaGriffin For a more obvious dark mode, disable this flag, and enable the chrome://flags/#force-dark-mode flag.

Also, If you are on linux, you will probably also want to set chrome://settings/appearance Theme to "Classic" instead of "GTK"

@WingofaGriffin
Copy link

Apologies, I wasn't clear in my first post.

I already found both of these flags on my own, but I think it would be smart to make them a part of the settings GUI or the extension for easier access for users who are afraid/don't know about the flags.

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

4 participants