Bug 365197

Summary:	add a notification settings for volume change events
Product:	[Unmaintained] plasma-pa	Reporter:	Till Schäfer <till2.schaefer>
Component:	applet	Assignee:	David Rosca <nowrep>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	max.schwarz, plasma-bugs-null
Priority:	NOR
Version First Reported In:	5.7.0
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:	http://commits.kde.org/plasma-pa/6814a96da412578a4be47c5643ff6ea13dfdf9e6	Version Fixed/Implemented In:
Sentry Crash Report:

Description Till Schäfer 2016-07-07 14:57:17 UTC

KMix plays a sound and shows an OSD whenever the volume changes by "volume up/down". In contrast, plasma-pa only shows the OSD, but plays no sound. The sound is helpful to get a feedback about the actual loudness of the sound device (its sometimes hard to guess how loud X% might be), especially if one wants to configure the volume before (and not while) playing some media.

As there are also peplle which may prefer a silent volume change, I would like to propose that  plasma-pa adds a volume change notification event that can be configured individually.

Reproducible: Always

Comment 1 Max Schwarz 2016-07-16 11:59:03 UTC

Also, I personally have no use for the OSD and would like to disable it. As it appears in the center of the screen (why?), it often occludes content I am watching.

Comment 2 David Rosca 2016-07-28 14:55:30 UTC

Git commit 6814a96da412578a4be47c5643ff6ea13dfdf9e6 by David Rosca.
Committed on 28/07/2016 at 14:54.
Pushed by drosca into branch 'master'.

Add option to enable volume feedback

Play "pop" sound when changing volume

Differential Revision: https://phabricator.kde.org/D2219

M  +1    -0    CMakeLists.txt
M  +3    -0    applet/contents/config/main.xml
M  +53   -22   applet/contents/ui/ConfigGeneral.qml
M  +4    -0    applet/contents/ui/ListItemBase.qml
M  +18   -0    applet/contents/ui/main.qml
A  +50   -0    cmake/FindCanberra.cmake
M  +2    -0    src/qml/CMakeLists.txt
M  +2    -0    src/qml/plugin.cpp
A  +83   -0    src/qml/volumefeedback.cpp     [License: LGPL]
A  +46   -0    src/qml/volumefeedback.h     [License: LGPL]

http://commits.kde.org/plasma-pa/6814a96da412578a4be47c5643ff6ea13dfdf9e6