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
Theming options for Gtk and Qt #71079
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we configure cursor themes for gtk as well?
I believe it's just another key/s in gtkrc.
Done, thanks! |
@worldofpeace @jtojnar merge please? |
@worldofpeace @jtojnar ping |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/configuring-system-wide-gtk-settings/5689/2 |
@gnidorah Could you expand the manual chapter "GTK/Qt themes" with a mention to this module and an example? |
@rnhmjoj Done, thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tested every option of gtk
and I can completely reproduce my current configuration with it.
Also the example works as expected. I tried another Qt style (arc-kde) but I can't get to work, it's probably my fault: I know nothing of Qt.
IMHO this has waited long enough, I hoped it would be in 20.03 already. Let's merge it.
@rnhmjoj Thank you! arc-kde doesn't provide native qt style, instead it provides settings for 3rd party theme engine (kvantum) |
I figured it was different somehow, that explains it.
Thank you |
👎 I feel like at least @jtojnar @ttuegel and myself should have approved on this before it was merged. I even expressed that to @gnidorah. With release management I've had almost no time to look at these things, which I'm sorry about. I probably should have at least mentioned that here also for everyone to see, instead of the mentions I made on IRC. |
As mentioned, it may look like it has waited and received improvements off feedback, but there really hasn't been much. |
Indeed, it's sad for something like this to go unnoticed. |
#80854. Could you open another PR for this? I will review it. |
It would also help, though I think you'd have to rewrite your history 😦, if the gtk module and the qt module weren't in the same PR. It just makes it easier to review, because they don't depend on each other. |
Is there any way to override theme colors declaratively? I'd like to replace some colors with custom ones I have defined to be used throughout my configuration in variables |
Hm, that only sets the the gtk theme, if I'm not mistaken. I was referring to changing the individual colors. For now I ended up doing it like this: https://github.com/pinpox/nixos-home/blob/1cefe28c72930a0aed41c20d254ad4d193a3fa37/gtk.nix#L11 |
@rycee Sure thing, go ahead. I have added MIT to the files, let me know if you have any questions |
@pinpox Thanks, I've added it now. |
Motivation for this change
In place of #70276
When I started to use NixOS at desktop I was wondered that there is no module to set Gtk and Qt widget and icon themes (in bare WM environment). Its no joy to use lxappearence, qt5ct and similar tools. These modules fix that issue by adding a way to configure themes declaratively.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @worldofpeace
TODO: ping jtojnar mid february