<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham 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/D26555">View Revision</a></tr></table><br /><div><div><p>Are we sure this is the right approach? In the Breeze theme, the icon is the same as the generic magnifying glass search icon, but in the Oxygen and Breeze theme, it's some kind of paint splatter, which as <a href="https://phabricator.kde.org/p/bruns/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@bruns</a> suggests, is not appropriate to use as a generic search icon.</p>

<p>How is anything still using this icon anyway? I thought milou switched to using the <tt style="background: #ebebeb; font-size: 13px;">search</tt> icon? This should fall back to the icon in the Breeze theme.</p>

<p>If we need a <tt style="background: #ebebeb; font-size: 13px;">search</tt> icon in the plasma theme, then we need to make sure it's semantically appropriate. For Breeze, it should be a line-art magnifying glass, for Oxygen, it should be a skeumorphic magnifying glass, etc.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R242 Plasma Framework (Library)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26555">https://phabricator.kde.org/D26555</a></div></div><br /><div><strong>To: </strong>The-Feren-OS-Dev, Plasma, VDG, ndavis, bruns, ngraham<br /><strong>Cc: </strong>ngraham, bruns, ndavis, davidre, kde-frameworks-devel, LeGast00n, GB_2, michaelh<br /></div>