<table><tr><td style="">filipf 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/D15999">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/D15999#368500" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D15999#368500</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><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/D15999#368499" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D15999#368499</a>, <a href="https://phabricator.kde.org/p/filipf/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@filipf</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>But there is a tangible problem here - imagine what the logout screen when using dark themes would look like if the circles were dark[;] They would be weird looking and would be superfluous.</p></div>
</blockquote>
<p>That's precisely why I wanted a subtle outline around the edge of the background circle, which earlier versions had. With that, a dark circle on a dark wallpaper didn't become muddy and fade into the wallpaper; it still looked crisp and handsome.</p></div>
</blockquote>
<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/D15999#368500" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D15999#368500</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><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/D15999#368499" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D15999#368499</a>, <a href="https://phabricator.kde.org/p/filipf/" style="
border-color: #f1f7ff;
color: #19558d;
background-color: #f1f7ff;
border: 1px solid transparent;
border-radius: 3px;
font-weight: bold;
padding: 0 4px;">@filipf</a> wrote:</div>
<div style="margin: 0;
padding: 0;
border: 0;
color: rgb(107, 116, 140);"><p>But there is a tangible problem here - imagine what the logout screen when using dark themes would look like if the circles were dark[;] They would be weird looking and would be superfluous.</p></div>
</blockquote>
<p>That's precisely why I wanted a subtle outline around the edge of the background circle, which earlier versions had. With that, a dark circle on a dark wallpaper didn't become muddy and fade into the wallpaper; it still looked crisp and handsome.</p></div>
</blockquote>
<p>I've made a <strong>very</strong> rough sketch (apologies for the roughness) to test things out. It's not that bad.</p>
<p><a href="https://phabricator.kde.org/F6448064" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6448064: logoutnostroke.png</a></p>
<p>Obviously it's a bit better when there's strokes:</p>
<p><a href="https://phabricator.kde.org/F6448068" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6448068: logoutwstroke.png</a></p>
<p>Though I have to admit I usually find strokes and outlines to be visually detrimental; in this case they also sort of overwhelm me because my brain is saying "there's too many lines". I also don't like how they're not of the same color as the icon itself, whereas if you make them white they will poke your eyes out.</p>
<p>But there is a more important thing white circles would have going for them - how obvious it is which logout option is selected (which was also one of the reasons behind adding circles). A dark circle having its opacity lowered on a dark background is just less effective than a white circle on a dark background. So tl;dr = I still think white circles with dark themes are a better choice in the case of the logout screen.</p>
<p>When it comes to SDDM, a lot of our problems stem from how uncustomizable it is... but what Phil said, if it could somehow detect the darkness of the wallpaper and adjust icons (maybe even other elements?) appropriately, that would be fantastic.</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/D15999">https://phabricator.kde.org/D15999</a></div></div><br /><div><strong>To: </strong>pstefan, ngraham, VDG, ndavis<br /><strong>Cc: </strong>ndavis, broulik, filipf, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>