<table><tr><td style="">mart 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/D19822">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D19822#439297" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D19822#439297</a>, <a href="https://phabricator.kde.org/p/hein/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@hein</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>A possible improvement to make here is to make the indicator visually react to hover by recoloring the icon, to teach that it's seperately interactive and will do something different than clicking elsewhere on the task button, so users get habituated not to click it accidentally. This should probably be done anyway (we're usally pretty triggerhappy on the hover feedback).</p></div>
</blockquote>

<p>If you want to recolor it dramatically indeed you would have to use kirigami.Icon (or add the same feature to iconitem, which is also fine)<br />
you would need to recolor both states, normal and hover, otherwise the normal state may be recolored the wrong color (in cases when the plasma theme doesn't follow system palette)</p>

<p>alternatively to use the standard mouseover effect, it can use the builtin highlight effect in iconitem made with kiconeffect (maybe if is too light, the default highlight effect in kiconeffects has to be reconsidered?)</p>

<p>weight-wise, kirigami.icon should be slightly ligher (eh, not too much :p) than iconitem and would like in the future (kf6?) to just have that one instead of the duplication plasmacore.iconitem and kirigami.icon</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19822">https://phabricator.kde.org/D19822</a></div></div><br /><div><strong>To: </strong>faridb, Plasma, VDG, ngraham, filipf, hein<br /><strong>Cc: </strong>mart, hein, filipf, mvourlakos, ngraham, broulik, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>