[Differential] [Commented On] D4845: Fix context menu button's icon size when on default DPI

[Chris Holland]

Zren added a comment.


  That's a good idea @broulik .
  
  Slider height for the Desktop themes I have installed (I used this script <https://gist.github.com/Zren/4864507f2eeddfe36c7496d59a8fa295>):
  
  - air: 20px
  - Arctica: 24px
  - Arezzo: 24px
  - arquetype-dark: 24px
  - Bise: 24px
  - breeze-alphablack: 24px
  - BreezeAlphaBlackNaked: 24px
  - damadamas: 24px
  - Freeze: 24px
  - **Fushigi: 12px <==**
  - Ghost: 24px
  - Glassified: 16px
  - Glassified: 16px
  - Glaze: 24px
  - Glaze: 24px
  - Helium: 24px
  - Helium: 24px
  - Interceptor: 24px
  - K10ne: 18px
  - K10neBlack: 18px
  - Mild: 24px
  - Professional: 24px
  - SimpleFlow_Modified: 24px
  - slim-glow: 24px
  - Sly: 24px
  - tragedy: 24px
  - Underworld: 16px
  - Volatile: 16px
  - Wave: 24px
  
  Only Fushigi seems to be <16px, but a number of sliders are 16px which means the icon only has 0-1px of padding.
  
  So we probably need to set `Layout.preferredHeight: Math.max(slider.height, units.iconSizes.small)` in the ToolButton?

REPOSITORY
  R115 Plasma Audio Volume Applet

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: Zren, subdiff, drosca, broulik
Cc: 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/20170228/a864dcd9/attachment-0001.html>