<table><tr><td style="">hein created this revision.<br />hein added a reviewer: Plasma.<br />Restricted Application added a project: Plasma.<br />hein requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D10223" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This patch does two things:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">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.</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">We set IconItem to do smooth scaling.</li>
</ul>

<p>While this results in lovely visual fidelity, I the performance<br />
impact is a concern. We're requesting twice as large thumbnails<br />
now, and we're doing more scaling work. However, thumbnail<br />
generation is async and doesn't slow down listing. There's also<br />
the possibility IconItem::smooth could have general performance<br />
impact unless things are smart enough not to scale when not<br />
necessary, which is hopefully the case.</p>

<p>BUG:376848</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>Plasma/5.12</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10223" rel="noreferrer">https://phabricator.kde.org/D10223</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containments/desktop/package/contents/ui/FolderItemDelegate.qml<br />
containments/desktop/package/contents/ui/FolderView.qml<br />
containments/desktop/plugins/folder/foldermodel.cpp</div></div></div><br /><div><strong>To: </strong>hein, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>