<table><tr><td style="">ndavis requested changes to this revision.<br />ndavis added a comment.<br />This revision now requires changes to proceed.
</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/D23464">View Revision</a></tr></table><br /><div><div><p>You're getting there.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">16px: needs 2px margins on the top and bottom</li>
<li class="remarkup-list-item">22px: needs 3px margins on the top and bottom</li>
<li class="remarkup-list-item">32px: needs to be lined up with the pixel grid and needs 2px margins on the top and bottom. You'd really be better off basing the background shape on another 32px icon and then applying the colors you want to use rather than trying to use a shrunken version of the the 64px background.</li>
</ul>

<p>In case you didn't see it before, there is an HIG and we recently improved the structure of the icon design section. <a href="https://hig.kde.org/style/icons/mimetype.html" class="remarkup-link" target="_blank" rel="noreferrer">https://hig.kde.org/style/icons/mimetype.html</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23464">https://phabricator.kde.org/D23464</a></div></div><br /><div><strong>To: </strong>alexde, VDG, ndavis, ngraham<br /><strong>Cc: </strong>trickyricky26, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>