[KDE/Mac] Review Request 126624: respect "ShowIconsOnButtons" in the Qt5/KF5 style

René J.V. Bertin rjvbertin at gmail.com
Mon Jan 18 16:48:09 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126624/
-----------------------------------------------------------

(Updated Jan. 18, 2016, 5:48 p.m.)


Review request for KDE Software on Mac OS X, KDE Frameworks and Yichao Yu.


Changes
-------

I blame the reviewboard software upgrade for messing up my patch; the patch copy I still had on disk wasn't empty at all.


Repository: qtcurve


Description
-------

Implementation details have changed in Qt5 that affect when icons were being shown in pushbuttons; as a result, many buttons were drawn with icons visible regardless the `ShowIconsInButtons` preference.

This patch implements a solution in the Qt5 style itself, which appears to be the only way to respect the preference both robustly (= on all buttons) and reversibly (i.e. icons will appear or disappear when toggling the setting without need to restart running applications).
Buttons that have only an icon but no text will continue to show the icon.

Note that `KGlobalSettings::showIconsOnPushButtons()` is deprecated and will have to be replaced with something that is not...


Diffs (updated)
-----

  qt5/style/qtcurve_api.cpp 44b2068 

Diff: https://git.reviewboard.kde.org/r/126624/diff/


Testing
-------

On OS X 10.9.5 and Linux, both with Qt 5.5.1 and KF5 5.17.0 installed into /opt/local


Thanks,

René J.V. Bertin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20160118/62f2a308/attachment.html>


More information about the kde-mac mailing list