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
