Only update mute state after volume change is signalled from PA
AbandonedPublic

Authored by drosca on Jan 15 2017, 10:05 AM.

Details

Reviewers
None
Group Reviewers
Plasma
Summary

There is no difference for normal streams where setVolume and setMute
are two different API calls, but it fixes issue with setting volume to
0 for ext-stream-restore streams (notification sounds) where there is
only one API call to change all stream properties.

Test Plan

Moving volume slider to 0 for Notification sounds in KCM now correctly
sets the volume to 0 and mute it. Functionality with other streams is unchanged.

Diff Detail

Repository
R115 Plasma Audio Volume Applet
Branch
mute-eventstream
Lint
No Linters Available
Unit
No Unit Test Coverage
drosca updated this revision to Diff 10173.Jan 15 2017, 10:05 AM
drosca retitled this revision from to Only update mute state after volume change is signalled from PA.
drosca updated this object.
drosca edited the test plan for this revision. (Show Details)
drosca added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptJan 15 2017, 10:05 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
drosca abandoned this revision.Feb 20 2017, 9:06 AM

Replaced with D4674