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

Phil Stefan noreply at phabricator.kde.org
Mon Oct 8 15:12:16 BST 2018


pstefan added a comment.


  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.

INLINE COMMENTS

> UserDelegate.qml:54
> +
> +        width: imageSource.width + 10
> +        height: width

I am not sure the 10px(?) padding are needed. The white circle is already a border around the image i.e. it's always shown, no matter the icon used. So this background circle is akin to drawing a dark rectangle behind the input field and button.

> UserDelegate.qml:58
> +
> +        color: "black"
> +        opacity: 0.5

If we hardcode this color we should use the same color as in the icons or vice versa. I used #232627 50% opacity.

REPOSITORY
  R120 Plasma Workspace

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

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


More information about the Plasma-devel mailing list