<table><tr><td style="">ngraham created this revision.<br />ngraham added reviewers: Frameworks, Dolphin, VDG.<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/D15866">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>An 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 <a href="https://phabricator.kde.org/T9740#162147" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T9740#162147</a> for examples.</p>

<p>This patch adjusts the calculation so that the emblem never takes up more then 18.75% of the total area of an icon, even at small sizes. As a result, the icons are less obscured by their emblems.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>[images go here]</p>

<p>[images depict the new hard drive icon introduced with <a href="https://phabricator.kde.org/D15853" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D15853</a>]</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R302 KIconThemes</div></div></div><br /><div><strong>BRANCH</strong><div><div>smaller-emblems-at-small-sizes (branched from master)</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>AFFECTED FILES</strong><div><div>src/kiconloader.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, Dolphin, VDG<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>