<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D19173">View Revision</a></tr></table><br /><div><div><p>I'm not in favor of this at all in its current form. Any tinting of the text with the background color under it reduces legibility. I'm as in favor of aesthetically appealing user interfaces as anyone, but not at the expense of usability.</p>

<p>A smaller degree of tint for the text <em>might</em> work, but then it just becomes closer to the base color and what's the point? Alternatively, tinting just the icon might work though.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19173">https://phabricator.kde.org/D19173</a></div></div><br /><div><strong>To: </strong>filipf, Kirigami, VDG, ngraham<br /><strong>Cc: </strong>plasma-devel, dkardarakos, apol, davidedmundson, mart, hein<br /></div>