D5144: Change the volume icon/mute button into a ToolButton

[Chris Holland]

Zren added a comment.


  > I've seen that, but that is unrelated to this change, so please move it into separate review.
  
  Sorry. I hit a wall on that bug and it drove me crazy. I just found out that changing `spacing: 0` to `spacing: 1` will cause things to layout correctly the first time.
  
  My theory is that clicking the button will change the ToolButton's svg, which probably causes the naturalHeight to be set, causing the ColumnLayout to recalculate the height of the rows. Whatever is loading wrong at initialization is corrected, causes the reflow, which makes the text shift.

REPOSITORY
  R115 Plasma Audio Volume Applet

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

To: Zren, #plasma, subdiff, drosca
Cc: mart, subdiff, drosca, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170403/37b14ded/attachment-0001.html>