<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/D23711">View Revision</a></tr></table><br /><div><div><p>We're supposed to use 6px margins on the top and bottom, making the visible part of the icon 52px tall. Bottom line shadows are normally supposed to be 1px tall. Other than that, it looks pretty good.</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/D23711">https://phabricator.kde.org/D23711</a></div></div><br /><div><strong>To: </strong>GB_2, VDG, ngraham, ndavis<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, VDG, LeGast00n, GB_2, michaelh, bruns<br /></div>