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