Zren added a comment.

  Looks good.
  
  F3126554: 2017-03-22___20-56-01.png <https://phabricator.kde.org/F3126554>
  
  There's one issue I want to point out that I dealt with, is that other 
instances of the widget won't be in the "raised" state (eg: other screens).
  
  Example: https://streamable.com/go1u7
  
  My solution was:
  
https://github.com/Zren/plasma-applets/blob/master/org.kde.plasma.volumewin7mixer/package/contents/ui/MixerItem.qml#L372-L379
  
    property int volume: PulseObject.volume
    onVolumeChanged: {
        ignoreValueChange = true;
        if (!mixerItem.isVolumeBoosted && PulseObject.volume > maximumValue) {
            mixerItem.isVolumeBoosted = true;
        }
        value = PulseObject.volume;
        ignoreValueChange = false;
    }

REPOSITORY
  R115 Plasma Audio Volume Applet

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

To: drosca, #plasma
Cc: Zren, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol

Reply via email to