<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Frameworks, Dolphin, VDG, broulik, cfeck.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />ngraham 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/D15071">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This patch improves and simplifies the criteria for whether or not to draw a frame and a shadow around an image's thumbnail. The old size-based detection code was unreliable and gave false positives as well as false negatives. It is replaced with a simple check for whether the image has an alpha channel, which not only automatically matches all icon files, but also non-icon raster images with transparency, which look better without the frame.</p>

<p>BUG: 258514<br />
FIXED-IN: 5.50</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Verified unchanged:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">SVG icons still have no frame:</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">JPEG and PNG images without transparency still have a frame:</li>
</ul>

<p>Verified changed:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Non-icon SVG and PNG images with transparency lose their frames. The difference is especially welcome for the cases where the previous set of criteris caused many false positives for folders full of mixed image formats and sizes. For example:</li>
</ul>

<p>Before:</p>

<p>After:</p>

<p>Before:</p>

<p>After:</p>

<p>Before:</p>

<p>After:</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>thumbnail-frame-refinement (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15071">https://phabricator.kde.org/D15071</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/filewidgets/kfilepreviewgenerator.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin, VDG, broulik, cfeck<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>