lxqt / lxqt-qtplugin

LXQt Qt platform integration plugin
https://lxqt.github.io
GNU Lesser General Public License v2.1
24 stars 14 forks source link

Made the palette configurable in `LXQtPlatformTheme` #55

Closed tsujan closed 4 years ago

tsujan commented 4 years ago

Only the window (= button) color can be configured for now — QPalette::QPalette(const QColor&) calculates the other colors automatically — but, if needed, more customization could be added later, although, IMHO, it is better not to confuse users with too many colors.

The default color is that of Fusion's window.

If an option is added to lxqt-config, users could change not only Fusion's look-and-feel but also Breeze's, without needing KDE's systemsettings5.

Note 1: This was started to compensate for the ugly color Qt 5.15 gave to Fusion. IMO, that's a Qt issue but we don't need to worry about it.

Note 2: I'll make another PR for lxqt-config after the current one is decided.

Closes https://github.com/lxqt/lxqt-qtplugin/issues/54

tsujan commented 4 years ago

Could you release lxqt-qtplugin 0.15.1 with this patch?

Actually, I wanted to ask about it. So, yes, after this is merged.

The (future) patch for lxqt-config could be in LXQt 0.16.0.

tsujan commented 4 years ago

@luis-pereira Thanks for the review! @yan12125 has tested it with 5.15.

@yan12125 I'll release lxqt-qtplugin 0.15.1 tomorrow.