<table><tr><td style="">ngraham edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-qdw37ckohyi3tpc/">(Show Details)</a><br />ngraham edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ixntoadpnyswohv/">(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/D15866">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">An</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">A single</span> icon's emblem can take up 25% of the total icon area in certain cases, which obscures enough of the icon that it may become hard to tell what it is. This is especially noticeable with 16x16 and 32x32 icons in Dolphin's Places Panel. See T9740#162147 for examples.<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
<br />
Because an icon can have up to four emblems, it is theoretically possible for a small icon with four emblems to show none of the icon itself, just emblems!</span><br />
<br />
This patch adjusts the calculation so that <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">a single</span> emblem never takes up more then 18.75% of the total area of an icon, even at small sizes. As a result, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the </span>icons are less obscured by their emblems.</div></div></div><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);">[images go here]</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Dolphin Places panel:<br />
<br />
Default: Small icons (16x16): {F6296114, size=full}<br />
<br />
Medium icons (22x22): {F6296116, size=full}<br />
<br />
Large icons (32x32): {F6296117, size=full}<br />
<br />
Huge icons (48x48): {F6296118, size, full}</span><br />
<br />
[images depict the new hard drive icon introduced with D15853]</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R302 KIconThemes</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15866">https://phabricator.kde.org/D15866</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin, VDG<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>