D8232: Add option to center shadow

[Michail Vourlakos]

mvourlakos added a comment.


  my previous screenshot was to show you the problem with the current implementation not to suggest new defaults...
  
  in order to demonstrate you the best in my liking I will give you two screenshots using the current breeze defaults.
  The first one is the default current breeze shadow, the second is my liking which has been produced by reducing the offset
  to half and changing line #643 of breezedecoration.cpp to:
  
    const int shadowOffset = qMax( 3*g_shadowSize/16, 2*Metrics::Shadow_Overlap/2 );
  
  Current:
  F5429749: screen_shadow_current.png <https://phabricator.kde.org/F5429749>
  
  Proposed:
  F5429752: screen_shadow_proposed.png <https://phabricator.kde.org/F5429752>

REPOSITORY
  R31 Breeze

REVISION DETAIL
  https://phabricator.kde.org/D8232

To: rpelorosso, #breeze, #vdg, hpereiradacosta
Cc: mvourlakos, 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/308347eb/attachment.html>