<table><tr><td style="">ndavis accepted this revision.<br />ndavis added a comment.<br />This revision is now accepted and ready to land.
</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/D27044">View Revision</a></tr></table><br /><div><div><p>LGTM</p>

<p>If you're not sure, use <tt style="background: #ebebeb; font-size: 13px;">scour</tt> because it's the least likely to screw anything up. As long as the code is formatted well and not filled with useless noise, it's fine if it isn't as small as it could be.</p>

<p>I optimized it a bit more with SVG Cleaner, used viewbox only for setting the canvas size and deleted a big invisible rectangle. Use this if you want: <a href="https://phabricator.kde.org/F8033944" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8033944: kate.svg</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R266 Breeze Icons</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27044">https://phabricator.kde.org/D27044</a></div></div><br /><div><strong>To: </strong>cullmann, ngraham, VDG, ndavis, tysontan<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>