<table><tr><td style="">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/D25340">View Revision</a></tr></table><br /><div><div><p>I like the background changes here and I think it makes logical sense to have a progression of changing background colors to indicate the status. However I'm not a huge fan of reducing the active indicator to just 2px. It's now so small that it becomes kind of superfluous. I think it would look better at the current size, and this would be more consistent with the highlight effects that <a href="https://phabricator.kde.org/p/ndavis/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ndavis</a> is working on, IIRC.</p>

<p>My view is that if we don't like the blue line being so visible, we should remove it entirely rather than making it so small that it feels like it wishes it didn't exist. :) Otherwise we should keep it where it is right now. I don't think there's anything wrong with its current size.</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/D25340">https://phabricator.kde.org/D25340</a></div></div><br /><div><strong>To: </strong>niccolove, VDG<br /><strong>Cc: </strong>ngraham, ndavis, kde-frameworks-devel, LeGast00n, GB_2, michaelh, bruns<br /></div>