<table><tr><td style="">drosca requested changes to this revision.<br />drosca added a comment.<br />This revision now requires changes to proceed.
</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/D5144" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I don't like it, the context menu button is enough hacks for one applet. Also it doesn't have spacing so it looks wrong compared to other ToolButtons (contex menu have spacing, but the icon size is still iconSizes.small).<br />
If we want to use ToolButtons in this way, it should be fixed in ToolButton.</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/D5144" rel="noreferrer">https://phabricator.kde.org/D5144</a></div></div><br /><div><strong>To: </strong>Zren, Plasma, subdiff, drosca<br /><strong>Cc: </strong>subdiff, drosca, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol<br /></div>