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

Reply via email to