Bug 410716 - The Volume Control at system tray play a plop sound with any changes
Summary: The Volume Control at system tray play a plop sound with any changes
Status: RESOLVED FIXED
Alias: None
Product: plasma-pa
Classification: Plasma
Component: applet (show other bugs)
Version: unspecified
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: David Rosca
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-08-08 08:29 UTC by S Y N C R E X
Modified: 2019-08-08 16:35 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.17.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description S Y N C R E X 2019-08-08 08:29:52 UTC
It is better to only play sound when the mouse click is released , not by any changes .
Comment 1 S Y N C R E X 2019-08-08 08:41:02 UTC
In fact, the best way to do this is when releasing a mouse click , looking for any media in play , if detected any media in play the Volume Controlمer does not need to play the plop sound  because at that time any media is in play user can be hear the media sound and at that time the plop sound is annoying .
Comment 2 Nate Graham 2019-08-08 15:24:57 UTC
There's a patch that fixes this: https://phabricator.kde.org/D22316
Comment 3 Nate Graham 2019-08-08 16:35:34 UTC
Git commit a420dd6be1a9a5b51c50074ce9d09b77a0a5cbbe by Nate Graham, on behalf of Kezi Olio.
Committed on 08/08/2019 at 16:35.
Pushed by ngraham into branch 'master'.

Avoid garbling the sound if the volume slider is moved fast

Summary:
The audio should not be garbled if you move fast the audio slider  (it distorts because it stops and plays the feedback sound every few milliseconds)

An alternative approach is to `return`; if the `ca_context_playing` says that the sound is still playing, but that gives less "resolution"
Related: bug 409525
FIXED-IN: 5.17.0

Reviewers: #plasma, drosca, davidedmundson, ngraham

Reviewed By: #plasma, davidedmundson, ngraham

Subscribers: anthonyfieroni, filipf, ngraham, davidedmundson, plasma-devel, kezik

Tags: #plasma

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

M  +4    -4    applet/contents/ui/ListItemBase.qml

https://commits.kde.org/plasma-pa/a420dd6be1a9a5b51c50074ce9d09b77a0a5cbbe