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