<table><tr><td style="">Zren added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D4751" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I did, there wasn't any change from when I tested it with <tt style="background: #ebebeb; font-size: 13px;">Layout.maximumHeight</tt>, because there's padding around the icon. Huh, I didn't post a screenshot, here:</p>

<p><a href="https://phabricator.kde.org/F2657863" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F2657863: 2017-02-27___14-51-23.png</a></p>

<p><a href="https://github.com/KDE/plasma-framework/blob/master/src/declarativeimports/plasmastyle/ToolButtonStyle.qml#L73" class="remarkup-link" target="_blank" rel="noreferrer">https://github.com/KDE/plasma-framework/blob/master/src/declarativeimports/plasmastyle/ToolButtonStyle.qml#L73</a></p>

<p>The height of the icon is based off the implicitHeight of the invisible label, which is controlled by the <tt style="background: #ebebeb; font-size: 13px;">control.font</tt>. However, adding <tt style="background: #ebebeb; font-size: 13px;">font.pixelSize: slider.height</tt> did nothing so Iunno how to manipulate that. It's why I've used the pattern I mentioned in <a href="https://github.com/Zren/plasma-applets/blob/master/org.kde.plasma.volumewin7mixer/package/contents/ui/MixerItem.qml#L350-L357" class="remarkup-link" target="_blank" rel="noreferrer">my widget</a>.</p>

<p>I didn't realize the mute button is also using the <tt style="background: #ebebeb; font-size: 13px;">IconItem { MouseArea { anchor.fill: parent } }</tt> pattern. Hum. Logging the <tt style="background: #ebebeb; font-size: 13px;">onHeightChanged</tt> prints this:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">qml: volumeIcon.height 24.740000000000002
qml: volumeIcon.height 19.987499999999997
qml: volumeIcon.height 20.2
qml: contextMenuButton.height 45
qml: contextMenuButton.height 24</pre></div>

<p>There's no easy way to log what the icon's height inside the <tt style="background: #ebebeb; font-size: 13px;">contextMenuButton</tt> is, but I print screened and counted the pixels. There's 7 pixels around the icon. So that means the icon is only 10px (probably the default font height?).</p>

<p><a href="https://phabricator.kde.org/F2657920" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F2657920: 2017-02-27___15-07-15.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R115 Plasma Audio Volume Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4751" rel="noreferrer">https://phabricator.kde.org/D4751</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>subdiff, Plasma, drosca, Zren<br /><strong>Cc: </strong>broulik, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>