mart added a comment.

  In D11205#222643 <>, @ngraham wrote:
  > I think the real error is deriving the button size from a Spacing value and 
not a Size value. We should fix that, or else we'll have to do the same thing 
if we ever change the spacing again.
  indeed, tough the idea was to size it depending on iconsizes and using the 
largespacing value as padding, which is a concept used a lot (would be space 
used outside the icon, so semantically would make sense).
  i think the patch would be correct, following this reasoning

  R169 Kirigami


To: icota, #kirigami, mart
Cc: ngraham, plasma-devel, #kirigami, apol, davidedmundson, mart, hein

Reply via email to