D25691: [KDecoration] Use QIcon::paint
Kai Uwe Broulik
noreply at phabricator.kde.org
Mon Dec 2 18:43:01 GMT 2019
broulik created this revision.
broulik added reviewers: Plasma, hpereiradacosta.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.
REVISION SUMMARY
Properly takes into account device pixel ratio. It's what Breeze decoration code is doing.
TEST PLAN
5.17?
Fixes broken icon positioning in the KCM with high dpi.
F7796584: Screenshot_20191202_194011.png <https://phabricator.kde.org/F7796584>
There's still something funky with the shadows but this patch at least fixes the most obvious blunder:
F7796582: Screenshot_20191202_193955.png <https://phabricator.kde.org/F7796582>
Decoration in KWin still looks the same.
REPOSITORY
R113 Oxygen Theme
REVISION DETAIL
https://phabricator.kde.org/D25691
AFFECTED FILES
kdecoration/oxygenbutton.cpp
To: broulik, #plasma, hpereiradacosta
Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191202/ca290605/attachment-0001.html>
More information about the Plasma-devel
mailing list