D4845: Fix context menu button's icon size when on default DPI

Chris Holland noreply at phabricator.kde.org
Mon Mar 6 02:04:01 UTC 2017


Zren added a comment.


  Ah, the `contextMenu.show(x, y)` changed to `contextMenu.show()`.
  I've got to comment out `roundToIconSize: false` in the mute "button" to test now since that probably requires a new version of frameworks.
  
  I changed the `slider.height` to `Layout.preferredHeight: units.iconSizes.small`, which makes the button 16px. Screenshot below.
  
  F2723094: 2017-03-05___20-47-54.png <https://phabricator.kde.org/F2723094>
  
  Also including a screenshot of smallMedium which is 22px since it looks closer to breeze's slider.height.
  
  F2723624: 2017-03-05___20-56-54.png <https://phabricator.kde.org/F2723624>

REPOSITORY
  R115 Plasma Audio Volume Applet

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

To: Zren, subdiff, broulik, drosca, aacid
Cc: aacid, davidedmundson, 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/20170306/72eece52/attachment.html>


More information about the Plasma-devel mailing list