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

[FR]: Support using the Qt style as defined by the system #1352

Closed
guihkx opened this issue Mar 22, 2024 · 6 comments
Closed

[FR]: Support using the Qt style as defined by the system #1352

guihkx opened this issue Mar 22, 2024 · 6 comments
Assignees
Labels
Component-GUI Status-Fixed Ticket is resolved. Type-Desktop-Integration Application badly integrates into used desktop environment. Type-Enhancement This is request for brand new feature.
Milestone

Comments

@guihkx
Copy link
Contributor

guihkx commented Mar 22, 2024

Brief description of the feature request

On Linux distributions based on KDE Plasma, a Qt style named Breeze is used by default.

Despite of that, on a fresh RSS Guard installation, the Qt style seems hardcoded to Fusion.

It'd be cool to introduce an option to use the system style instead, similar to what is done with the icon theme:

image

@guihkx guihkx added the Type-Enhancement This is request for brand new feature. label Mar 22, 2024
@guihkx
Copy link
Contributor Author

guihkx commented Mar 22, 2024

I played around trying to implement this, but since I have no GUI expertise it'd be tricky for me to implement something actually useful.

I was able to access the name of system style with qApp->style()->objectName() (tested only on Linux with Qt 5.15). On Plasma 6, that method returns breeze.

@martinrotter
Copy link
Owner

I will test with some KDE-based VM.

@martinrotter
Copy link
Owner

Building right now. Now default style should default to system style, also added "system style" explicit item to settings. Feel free to test.

@martinrotter martinrotter added this to the 4.6.7 milestone Mar 25, 2024
@martinrotter martinrotter added Component-GUI Status-Fixed Ticket is resolved. Type-Desktop-Integration Application badly integrates into used desktop environment. labels Mar 25, 2024
@martinrotter
Copy link
Owner

Make sure to remove proper key from settings to test "vanilla" default.

@martinrotter
Copy link
Owner

image

@guihkx
Copy link
Contributor Author

guihkx commented Mar 25, 2024

Just tested it, works perfectly. Thank you very much once again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component-GUI Status-Fixed Ticket is resolved. Type-Desktop-Integration Application badly integrates into used desktop environment. Type-Enhancement This is request for brand new feature.
Projects
None yet
Development

No branches or pull requests

2 participants