<table><tr><td style="">ngraham edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-t43pag66qg7ihec/">(Show Details)</a>
</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>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Verified unchanged</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">== PNG with no alpha channel ==<br />
Previously some would get frames and some wouldn't, depending on their dimensions. Now all do.<br />
<br />
Before:<br />
{F6214978}<br />
<br />
After:<br />
{F6214988}<br />
<br />
<br />
<br />
== PNG with alpha channel ==<br />
Previously some would get frames and some wouldn't, depending on their dimensions. Now none do.<br />
<br />
Before:<br />
{F6214261}<br />
<br />
After</span>:<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- SVG icons still have no frame: </span>{F62142<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">57}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">62}</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- JPEG and PNG images without transparency still have a frame: </span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Before:<br />
</span>{F6214<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">258}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">955}</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Verified changed:</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">After:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- Non-icon SVG and PNG images with transparency lose their frames. The difference is especially welcome for the cases where the previous set of criteria caused many false positives for folders full of mixed image formats and sizes. For example:</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F6214956}</span><br />
<br />
Before:<div style="padding: 8px 0;">...</div>{F6214260}<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
<br />
Before:<br />
{F6214264}<br />
<br />
After:<br />
{F6214265}<br />
<br />
<br />
<br />
== JPEG ==<br />
Previously some would get frames and some wouldn't, depending on their dimensions. Now all do.<br />
<br />
Before:<br />
{F6214939}<br />
<br />
After:<br />
{F6214940}<br />
<br />
<br />
<br />
== SVG ==<br />
Previously some would get frames and some wouldn't, depending on their dimensions. Now they only do if they have transparency.</span><br />
<br />
Before:<br />
{F6214<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">261}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">943}</span><br />
<br />
After:<br />
{F6214<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">262}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">944}</span><br />
<br />
Before:<br />
{F6214<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">264}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">977}</span><br />
<br />
After:<br />
{F6214<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">265}</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">968}</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</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>To: </strong>ngraham, Frameworks, Dolphin, VDG, broulik, cfeck<br /><strong>Cc: </strong>abetts, bruns, kde-frameworks-devel, michaelh, ngraham<br /></div>