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

[Roman Gilg]

subdiff added subscribers: drosca, subdiff.
subdiff accepted this revision.
subdiff added a comment.
This revision is now accepted and ready to land.


  Looks very nice. I also don't think that the hover line looks weird. It's just a little bit stupid, that for these small buttons we always have to use the workaround with the IconItem. Can't we introduce a parameter in ToolButton to reduce the margins from its native icon? But this is unrelated to this patch. So accepted from my side. @drosca ?

REPOSITORY
  R115 Plasma Audio Volume Applet

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

To: Zren, #plasma, subdiff
Cc: 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/20170323/d7862ba8/attachment.html>