<table><tr><td style="">colomar 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/D2314" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Sorry for not replying earlier.<br />
There is nothing that speaks against offering this context menu. However, all the actions in the context menu must be available in the main UI as well. Due to discoverability reasons, context menus must never be the only way to access an action.<br />
So, this patch is good to go, but there needs to be a way to do these things without the context menu as well.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAPA Plasma Audio Volume Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2314" rel="noreferrer">https://phabricator.kde.org/D2314</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>Zren, Plasma, Plasma: Design<br /><strong>Cc: </strong>davidedmundson, drosca, colomar, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>