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

Reply via email to