drosca added a comment.

  In https://phabricator.kde.org/D2314#58816, @Zren wrote:
  
  > All I did was copy the Menu widget's pattern (sorta) since it was the first 
example I could find.
  
  
  Yeah, PlasmaComponents.ContextMenu sucks but if possible, I would prefer the 
declarative approach too. If not, then fine, but at least replace the 
Qt.createQmlObject with Component.createObject.
  
  > Not sure if VDG has talked about this yet, but here's a few things that 
could be added on top of the this context menu:
  > 
  > - Toggle Volume Boost (between 100% and 150%) so maximum isn't always 150%.
  > - Selecting "Default [device]" can move all the current app streams to that 
device.
  
  Sure, if we have a context menu I am not against adding a few more features 
to it. However, the main question for VDG is if we may actually have the 
context menu at all.

REPOSITORY
  rPLASMAPA Plasma Audio Volume Applet

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

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

To: Zren, #plasma, #plasma:_design
Cc: davidedmundson, drosca, colomar, plasma-devel, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas

Reply via email to