<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/D16031">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/D16031#339882" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D16031#339882</a>, <a href="https://phabricator.kde.org/p/broulik/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@broulik</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>Again: Can we just use shadows behind the logout buttons and avatar like we have for the clock?</p></div>
</blockquote>
<p>We can't do only a shadow for the avatar since it gives a shadow to the outline of the icon and not the shader effect, which looks bad. If we add a shadow behind the avatar (which I would be open to), the circle background I've added is mandatory.</p>
<p>For the buttons, it's a bit better, but I don't think <em>just</em> shadows behind the current icons will be enough:<br />
<a href="https://phabricator.kde.org/F6315209" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6315209: Drop shadows behind original icons, default background.png</a><br />
<a href="https://phabricator.kde.org/F6315210" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6315210: Drop shadows behind original icons, challenging background.png</a></p>
<p>Shadows behind the new circle-background icons do work pretty well though, and improve the contrast to good levels.:<br />
<a href="https://phabricator.kde.org/F6315212" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6315212: Drop shadows behind round icons, default background.png</a><br />
<a href="https://phabricator.kde.org/F6315214" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6315214: Drop shadows behind round icons, challenging background.png</a></p>
<p>Maybe that reveals that the icons' translucent background should be less translucent, or even opaque, and then we wouldn't need a shadow for them.</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/D16031">https://phabricator.kde.org/D16031</a></div></div><br /><div><strong>To: </strong>ngraham, VDG, Plasma<br /><strong>Cc: </strong>abetts, pstefan, broulik, rikmills, filipf, rooty, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart<br /></div>