-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
qt5ct module: expose qtstyleplugins #26761
Conversation
If I use KDE, I would like to have the whole thing working and not have to care about details and features like this. Can you make it such that by default this feature is also activated in that case? Regardless, of this feature request, this can be merged. |
@0xABAB Sorry, can't get it. qt5ct tool is intended for thoose, who don't use KDE. In KDE you set theming options including Qt/Gtk uniform look via KDE's System Settings. |
@gnidorah In that case I have no comments. |
Maybe somebody could merge this PR. It is becoming too old. |
@@ -30,4 +30,5 @@ in { | |||
|
|||
}; | |||
|
|||
meta.maintainers = [ maintainers.gnidorah ]; |
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.
nitpick: please use = with maintainers; [ gnidorah ];
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.
@peterhoeg Done, thanks!
@ttuegel is doing major work on the KDE/qt infrastructure right now - once he's done, I'll merge this. |
@peterhoeg Thank you! |
Motivation for this change
Add qtstyleplugins as runtime dependency, so GTK2 theme can be choosen as Qt 5 theme, like it was possible with Qt 4's qtconfig utility.
https://wiki.archlinux.org/index.php/Uniform_look_for_Qt_and_GTK_applications#QGtkStyle
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)