D16031: [SDDM theme] remove blur and increase UI contrast so that it's not required

Root noreply at phabricator.kde.org
Tue Oct 9 13:30:00 BST 2018


rooty added a comment.


  In D16031#339180 <https://phabricator.kde.org/D16031#339180>, @pstefan wrote:
  
  > As other have pointed out the shadows for the buttons and username look out of place. Do they use the same radius/spread as those of the clock?
  >
  > I think blur is still needed. If you look at the picture of the snow-y landscape the contrast suffers quite a bit because of the noisy background. In my mock-up test the smoothing of the blur helped quite in such cases.
  
  
  
  
  In D16031#339559 <https://phabricator.kde.org/D16031#339559>, @ngraham wrote:
  
  > I see what folks mean about the background circle for the avatar being too big. I'll reduce it, but IMHO it the "border" needs to be there, even if it's small. Compare the following images on a background that really highlights the need:
  >
  > No border: F6313303: Without border.png <https://phabricator.kde.org/F6313303>
  >
  > 2px border: F6313305: With border.png <https://phabricator.kde.org/F6313305>
  >
  > To my eyes there is no question, the second one looks better.
  >
  > I can also tweak the shadows Here's how it would look with the above change plus the button and username shadows tweaked to better match the clock's shadows: F6313297: Screenshot_20181008_193916.png <https://phabricator.kde.org/F6313297>
  
  
  circle border around avatar = it looks great in the last image, flush with the avatar
  shadows = can you make them more diffuse please and post it again? thanks

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D16031

To: ngraham, #vdg, #plasma
Cc: abetts, pstefan, broulik, rikmills, filipf, rooty, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181009/f0ab941d/attachment.html>


More information about the Plasma-devel mailing list