Skip to content
This repository has been archived by the owner on Jun 2, 2024. It is now read-only.

fix: trying to make theme work universally #261

Closed
wants to merge 2 commits into from
Closed

Conversation

nullishamy
Copy link
Contributor

Might close #258

Honestly at this point I don't even know what's happening.

GTK3 doesn't like it when you use -dark on dark themes, you must use -Dark for it to apply properly.
This does not apply to GTK4.

Copy link
Contributor

@sgoudham sgoudham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming this will work fine on my system but can we please add some documentation in our README.md surrounding this?

This is probably like our 7th time changing it and it's just because knowledge isn't being shared from contributor to contributor and/or written down. I think we finally have a good idea of what is happening on different GTK versions now as we've stopped relying so heavily on Colloid.

@nullishamy
Copy link
Contributor Author

I have no idea the semantics are for this

@sgoudham
Copy link
Contributor

sgoudham commented May 28, 2024

Honestly this is a good start:

GTK3 doesn't like it when you use -dark on dark themes, you must use -Dark for it to apply properly.
This does not apply to GTK4.

We need to start documenting and accumulating knowledge on it because of how odd it is. People might see that and offer up more info.

@nullishamy
Copy link
Contributor Author

I'd rather just wait and find out from the GTK people what's going on, opened an issue over there @ https://gitlab.gnome.org/GNOME/gtk/-/issues/6741

@sgoudham
Copy link
Contributor

sgoudham commented May 29, 2024

My testing on Pop!_OS 22.04 LTS on GNOME version 42.9 on X11:

-Dark: this suffix correctly themes every pane but sets the system theme incorrectly to Light mode.
-dark: this suffix does not theme every pane but does set the system theme correctly to Dark mode

@nullishamy
Copy link
Contributor Author

nullishamy commented May 29, 2024

My further testing on Fedora Workstation (40) (VM)

Fedora workstation does not care about the suffix, only about the presence of gtk-dark.css. the system theme does not impact our gtk theme at all.

GNOME 46 on Wayland

@nullishamy
Copy link
Contributor Author

dracula/gtk#264 (comment) A secondary report of the Pop!OS issue

@nullishamy nullishamy changed the title fix: -Dark, not a joke fix: trying to make theme work universally May 29, 2024
@nullishamy
Copy link
Contributor Author

See #262.

@nullishamy nullishamy closed this Jun 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GTK3 apps are not resolving the theme when the -dark suffix is applied
2 participants