<table><tr><td style="">trickyricky26 requested changes to this revision.<br />trickyricky26 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/D24959">View Revision</a></tr></table><br /><div><div><p>On the technical side, we usually convert everything to paths, and even though I can't imagine Qt SVG Renderer would have a problem with <tt style="background: #ebebeb; font-size: 13px;">circle</tt>, it might be better to convert the background to a path.</p>

<p>As for the design, I think this can be improved. My suggestion would basically be to use the magnifying glass from the Kfind icon, which IMO fits better with Breeze because of the thinner lines and the longer handle, which make the symbol feel more balanced.<br />
Additionally, maybe you could try adding the Breeze-typical 45° shadow, which is usually used in these symbol-on-background icons, although it might not look right with the magnifying glass extending in the same direction.</p>

<p>Overall I think this will be a nice addition.</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/D24959">https://phabricator.kde.org/D24959</a></div></div><br /><div><strong>To: </strong>astippich, VDG, ngraham, trickyricky26<br /><strong>Cc: </strong>trickyricky26, bruns, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham<br /></div>