D8232: Add option to center shadow
Henrik Fehlauer
noreply at phabricator.kde.org
Sat Oct 14 11:05:21 UTC 2017
rkflx added a comment.
FWIW, I would love this too. IMHO this should even be the default (but is obviously something for the https://phabricator.kde.org/tag/vdg/ to decide). Reasons:
- The shadow not only adds a spatial dimension to the visuals, it also is important in separating the contents of cascaded windows. Currently, this second function is not achieved for the left and top borders.
- It would be consistent with GNOME, Windows (all borders) and MacOS (all borders except the top).
Patch LGTM.
However, I noticed a small inconsistency (probably just food for thought and not something for this patch): Shadows of menus do not respect this, they are still offset. While that's the behaviour on Windows too, on MacOS menus have shadows on all three borders.
REVISION DETAIL
https://phabricator.kde.org/D8232
To: rpelorosso, #breeze, #vdg
Cc: rkflx, ngraham, davidedmundson, mart, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171014/73ae42c1/attachment.html>
More information about the Plasma-devel
mailing list