<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/D17419">View Revision</a></tr></table><br /><div><div><p>I noticed that the 24px version is now aligned to the grid, but the margins aren't right. The 16 and 22px versions are still not aligned to the grid.</p>

<p>The correct margin size for 24px is 4px on each side, but left and right margins can be exceeded if necessary. The 24px size is really only for applications that require 24px icons, such as some GTK applications. Do you actually need to use this size? There is no issue with adding it as long as it's correct, but it is slightly more work for you.</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/D17419">https://phabricator.kde.org/D17419</a></div></div><br /><div><strong>To: </strong>GB_2, Breeze, VDG, Falkon, ngraham, ndavis<br /><strong>Cc: </strong>ndavis, filipf, ngraham, VDG, kde-frameworks-devel, Breeze, alexde, IohannesPetros, trickyricky26, michaelh, crozbo, firef, bruns, skadinna, aaronhoneycutt, mbohlender<br /></div>