broulik created this revision. broulik added reviewers: Frameworks, dfaure, ngraham, pali, vonreth, antlarr. broulik requested review of this revision.
REVISION SUMMARY Prefer downsampling over upscaling TEST PLAN Dolphin requested 128px thumbnail here. Wine executables have 32, 48, and 256 icons. It would pick the 48 one: F6132594: Screenshot_20180723_204523.png <https://phabricator.kde.org/F6132594> Afterwards it would use the 256 ones F6132596: Screenshot_20180723_204442.png <https://phabricator.kde.org/F6132596> The effect isn't neccessarily as noticeable with every icon and I hardly have any "real" windows EXE files around to test :) Algo is basically copied from Plasma's wallpaper selection REPOSITORY R320 KIO Extras REVISION DETAIL https://phabricator.kde.org/D14308 AFFECTED FILES thumbnail/icoutils_common.cpp To: broulik, #frameworks, dfaure, ngraham, pali, vonreth, antlarr
