[Differential] [Request, 4 lines] D4138: Only update mute state after volume change is signalled from PA
David Rosca
noreply at phabricator.kde.org
Sun Jan 15 10:05:06 UTC 2017
drosca created this revision.
drosca added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION 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.
REPOSITORY
R115 Plasma Audio Volume Applet
BRANCH
mute-eventstream
REVISION DETAIL
https://phabricator.kde.org/D4138
AFFECTED FILES
applet/contents/ui/ListItemBase.qml
src/kcm/package/contents/ui/VolumeSlider.qml
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: drosca, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170115/72bf150c/attachment.html>
More information about the Plasma-devel
mailing list