<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/D27445">View Revision</a></tr></table><br /><div><div><p>Technical review:<br />
This patch has hardcoded colors in it. Gotta fix that.</p>

<hr class="remarkup-hr" />

<p>Visual review:<br />
<a href="https://phabricator.kde.org/F8107681" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F8107681: Screenshot_20200216_154612.png</a></p>

<p>Personally I don't really see the point of it. This patch makes the line so thin that it's practically not even there anymore. At that point, we might as well just remove it. If the line is going to be there, I think it feels better with some visual weight.</p>

<p>I won't block this patch if everyone loves the proposed change though; that's just my opinion. :)</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/D27445">https://phabricator.kde.org/D27445</a></div></div><br /><div><strong>To: </strong>niccolove, VDG<br /><strong>Cc: </strong>ngraham, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, bruns<br /></div>