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