D8232: Add option to center shadow
Hugo Pereira Da Costa
noreply at phabricator.kde.org
Sat Oct 14 13:29:35 UTC 2017
hpereiradacosta added a comment.
In https://phabricator.kde.org/D8232#155232, @rkflx wrote:
> 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.
This is not true
There is a shadow at the top and at the left. It is simply not as pronounced as the one at the bottom and the right. Also, the shadow strength (not its centering) is already configurable.
> - It would be consistent with GNOME, Windows (all borders) and MacOS (all borders except the top).
This is not a strong enough reason, imho
> 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.
REPOSITORY
R31 Breeze
REVISION DETAIL
https://phabricator.kde.org/D8232
To: rpelorosso, #breeze, #vdg, hpereiradacosta
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/6f4e3077/attachment.html>
More information about the Plasma-devel
mailing list