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