D10532: [Information Panel] Adjust painting icons for high dpi scaling

Kai Uwe Broulik noreply at phabricator.kde.org
Thu Feb 15 10:10:12 GMT 2018


broulik created this revision.
broulik added reviewers: Dolphin, elvisangelaccio.
broulik requested review of this revision.

REVISION SUMMARY
  Instead of creating our own `KIconLoader` (which takes ~5ms here) just use `QIcon::fromTheme` which already does everything right.
  
  BUG: 390488

TEST PLAN
  Multi-selection before
  F5712712: Screenshot_20180215_110351.png <https://phabricator.kde.org/F5712712>
  Multi-selection after
  F5712713: Screenshot_20180215_110400.png <https://phabricator.kde.org/F5712713>
  
  Search before
  F5712716: Screenshot_20180215_110410.png <https://phabricator.kde.org/F5712716>
  Search after
  F5712717: Screenshot_20180215_110414.png <https://phabricator.kde.org/F5712717>
  
  Before
  F5712714: Screenshot_20180215_110331.png <https://phabricator.kde.org/F5712714>
  After
  F5712715: Screenshot_20180215_110321.png <https://phabricator.kde.org/F5712715>
  
  Overlays are also correctly painted with D9814 <https://phabricator.kde.org/D9814>

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D10532

AFFECTED FILES
  src/panels/information/informationpanelcontent.cpp

To: broulik, #dolphin, elvisangelaccio
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180215/62f69c85/attachment.htm>


More information about the kfm-devel mailing list