D10584: fix double spacing bug

Hugo Pereira Da Costa noreply at phabricator.kde.org
Fri Feb 16 16:20:44 UTC 2018


hpereiradacosta added a comment.


  Thanks for the patch.
  I "think" one must also do a similar fix in "Style::menuItemSizeFromContents".
  It seems to me that there also the line 2728 
  leftColumnWidth += Metrics::MenuItem_ItemSpacing;
  should also be conditionned with "if iconWidth > 0"
   Can you double check ?

REPOSITORY
  R31 Breeze

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

To: zzag, #breeze, #vdg, hpereiradacosta
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180216/eb6103b0/attachment.html>


More information about the Plasma-devel mailing list