<table><tr><td style="">lavender added a comment.
</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/D22653">View Revision</a></tr></table><br /><div><div><p>Functionally it looks good to me, there are a few improvements/optimizations that can be made for example 16 and 22 have unnecessary attributes such as:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">fill="currentColor" style="fill:currentColor;</pre></div>

<p>And the 32 one has a transform that can be applied:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);"><svg xmlns="http://www.w3.org/2000/svg" id="svg938" width="54" height="54"><style id="current-color-scheme">.ColorScheme-Text{color:#232629}</style><path fill="currentColor" stroke-width="2" d="M27 9V7h2v2zm0 6v-2h2v2zm-4 2v-4h2v4zm24 24v2h-2v-2zm2-2v2h-2v-2zm-8 6v2h-4v-2zm-2 0v2h-2v-2zm0 4v2h-2v-2zm4-2v4h-2v-4zm8 4h-4v-2h4zm-4-4v4h-2v-4zm2-2v2h-6v-2zm2-4v4h-2v-4zm-18 2h2v2h-2zm2 2h2v2h-2zm-4 4v-2h6v2zm12-14h-2v-2h2zm0-2h-4v-2h4zm-6 0h-2v-2h2zm-6 2h2v4h-2zm0 4h2v2h-2zm20-12h-2v-4h2zm0-2h-2v-2h2zm-2-2h-6v-2h6zm-2 2h-2v-2h2zm2 4h-4v-2h4zm-4 8v-4h2v4zm4-4h-4v-2h4zm2 2h-2v-6h2zm0 4h-4v-2h4zm-16-8h-2v-4h2zm4-8h-2v-2h2zm-2 4h-4v-4h4zm2 4h-2v-2h2zm0-2v-4h2v4zm4-4h-4v-2h4zm2 2h-2v-2h2zm0 4h-4v-2h4zM13 21h2v4h-2zm10 8h2v2h-2zm-2-2h2v2h-2zm0 22h6v2h-6zm10 0h2v2h-2zm-10-4h2v2h-2zm2 2h2v2h-2zm4-4v4h-2v-4zm0 4h2v2h-2zm2 0v-8h2v8zm-8-8h8v2h-8zm14 2h6v2h-6zm6 2v-8h2v8zm0-6h-6v-2h6zm-4-2v6h-2v-6zm-10-2v4h-2v-4zm-4 4h4v2h-4zm-8-8h4v2h-4zm6 2h4v2h-4zm10-2v-2h2v2zm-4 4v-4h2v4zm2 2v-8h2v8zM25 5V3h2v2zm0 8v-2h2v2zM9 29h2v2H9zm14-8h4v2h-4zm-6 0h4v2h-4zm2 2h4v2h-4zm-4 2h14v2H15zm-4 0h4v4h-4zm-2-4h2v2H9zm0 2v4H7v-4zm-4 4h4v2H5zm-2-2h2v2H3zm0 6h8v2H3zm0-10h4v2H3zm24-2v-2h2v2zm6 0H21v2h12zm-12 0v-8h2v8zm8-2v-2h2v2zm2 4V11h2v10zm-2-10V9h2v2zm2-2V7h2v2zm-2-2V5h2v2zm0-2V3h4v2zm-8 4V5h2v4zm2 2V3h2v8zM7 47v-8h8v8zm-4 4v-2h16v2zm16 0h-2V35h2zm0-16v2H3v-2zM3 35h2v16H3zm36-20V7h8v8zm-4 4v-2h16v2zm16 0h-2V3h2zm0-16v2H35V3zM35 3h2v16h-2zM7 15V7h8v8zm-4 4v-2h16v2zm16 0h-2V3h2zm0-16v2H3V3zM3 3h2v16H3z" class="ColorScheme-Text"/></svg></pre></div>

<p>I used <a href="https://github.com/svg/svgo" class="remarkup-link" target="_blank" rel="noreferrer">svgo</a> to make these optimizations.</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/D22653">https://phabricator.kde.org/D22653</a></div></div><br /><div><strong>To: </strong>mbruchert, Breeze, ndavis<br /><strong>Cc: </strong>lavender, ngraham, broulik, ndavis, kde-frameworks-devel, LeGast00n, sbergeron, michaelh, bruns<br /></div>