<table><tr><td style="">broulik 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/D9227" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><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);">to asign the icon to the clear button</pre></div></blockquote>

<p>in principle it affects all action buttons, e.g. the "reveal password" button, it's not limited to the clear icon.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Would it not be better to file a bug and send a patch upstream ?</p></blockquote>

<p>Probably. I'll look into Qt code. From what I can tell in <tt style="background: #ebebeb; font-size: 13px;">QCommonStyle</tt> for <tt style="background: #ebebeb; font-size: 13px;">QIcon::Selected</tt> they tint the color slightly blue (with highlight color) whereas for <tt style="background: #ebebeb; font-size: 13px;">QIcon::Active</tt> they just don't do anything. Perhaps if I patch it to have it slightly highlight the active icon there (with <tt style="background: #ebebeb; font-size: 13px;">HighlightText</tt> instead of <tt style="background: #ebebeb; font-size: 13px;">Highlight</tt>?) it'll have some feedback for the button so we can drop the wrong icon usage here.</p>

<p>Not sure they'll like it, we'll see. In Qt styles there's one "hover" PNG for <tt style="background: #ebebeb; font-size: 13px;">standardbutton-closetab-hover-16.png</tt> whereas all others don't have a dedicated graphic. Also, you don't want hover icon states in e.g. Windows 95 style. Or I just dump a patch for removing the wrong usage in <tt style="background: #ebebeb; font-size: 13px;">QLineEditIconButton</tt> and hope a good explanation will convince them ;)</p>

<p>Either way I'll also investigate the <tt style="background: #ebebeb; font-size: 13px;">polish()</tt> hack.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R31 Breeze</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9227" rel="noreferrer">https://phabricator.kde.org/D9227</a></div></div><br /><div><strong>To: </strong>broulik, Plasma, VDG, hpereiradacosta<br /><strong>Cc: </strong>ngraham, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>