<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/D19370">View Revision</a></tr></table><br /><div><div><p>Oh one more thing: the <tt style="background: #ebebeb; font-size: 13px;">ActionButton</tt> component is also used in the logout screen. Does this look good there too when the background is dark, or does it become kind of muddy?</p>

<p>The way we solved this in the old version was by adding a light-colored outline around the circle to provide contrast when the background is also dark. I'd recommend that we do that here too, or else make the background circle optional and omit it on the logout screen when its background is dark.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19370">https://phabricator.kde.org/D19370</a></div></div><br /><div><strong>To: </strong>filipf, VDG, ngraham, rooty, Plasma<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>