D10223: Improve preview thumbnail quality
Eike Hein
noreply at phabricator.kde.org
Thu Feb 1 09:13:22 UTC 2018
hein created this revision.
hein added a reviewer: Plasma.
Restricted Application added a project: Plasma.
hein requested review of this revision.
REVISION SUMMARY
This patch does two things:
- Instead of giving the KAbstractViewAdapter the actual icon size, we overprovision by giving it the size times two. This is because while we're deriving the grid view cell size from the icon size setting, it's not the actual size the thumbnails will be displayed at. Our IconItem is sized to almost fill the cell, and for image sources with non-square aspect ratios IconItem doesn't scale to the next icon size.
- We set IconItem to do smooth scaling.
While this results in lovely visual fidelity, I the performance
impact is a concern. We're requesting twice as large thumbnails
now, and we're doing more scaling work. However, thumbnail
generation is async and doesn't slow down listing. There's also
the possibility IconItem::smooth could have general performance
impact unless things are smart enough not to scale when not
necessary, which is hopefully the case.
BUG:376848
REPOSITORY
R119 Plasma Desktop
BRANCH
Plasma/5.12
REVISION DETAIL
https://phabricator.kde.org/D10223
AFFECTED FILES
containments/desktop/package/contents/ui/FolderItemDelegate.qml
containments/desktop/package/contents/ui/FolderView.qml
containments/desktop/plugins/folder/foldermodel.cpp
To: hein, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180201/7aebb056/attachment.html>
More information about the Plasma-devel
mailing list