<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/D11928">View Revision</a></tr></table><br /><div><div><p>I love it! This solves the problems we set out to solve and it looks fantastic.</p>

<p>Just a few remaining comments:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">The commit message and summary need to be updated to reflect the latest changes</li>
<li class="remarkup-list-item">I'm still not a huge fan of how the interactive UI elements disappear after a little while. I think that will catch people by surprise. If we have to keep that, I would raise the timeout to more like 30 seconds or more.</li>
<li class="remarkup-list-item">Users have been pretty well trained over the last 30 years to fling the mouse around or bang on the keyboard to "wake up" their computers, so not showing the interactive UI elements before some activity is detected doesn't present a usability problem here in the lock screen. But we can't implement this exact same design on the login screen since users expect for the UI to be visible by default there, which brings us back to the original problem of insufficient legibility. We can continue this discussion in the follow-up SDDM theme patch though, but it's something to keep in mind.</li>
</ul></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/D11928">https://phabricator.kde.org/D11928</a></div></div><br /><div><strong>To: </strong>mart, Plasma, VDG, ngraham<br /><strong>Cc: </strong>richardbowen, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>