Bug 458977 - Volume sliders go beyond 100% even if 'Raise maximum volume' is unchecked in plasma-pa applet
Summary: Volume sliders go beyond 100% even if 'Raise maximum volume' is unchecked in ...
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_pulseaudio (show other bugs)
Version: master
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on: 440846
Blocks:
  Show dependency treegraph
 
Reported: 2022-09-10 20:56 UTC by Patrick Silva
Modified: 2022-11-03 23:54 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.27


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2022-09-10 20:56:02 UTC
STEPS TO REPRODUCE
1. make sure 'Raise maximum volume' option of plasma-pa applet is unchecked
2. open Audio KCM
3. 

OBSERVED RESULT
the volume sliders can be used to set volume beyond 100%

EXPECTED RESULT
maximum volume should be 100% when 'Raise maximum volume' is unchecked

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.25.80
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6
Graphics Platform: Wayland
Comment 1 ratijas 2022-09-11 11:23:28 UTC
Not exactly a bug, and fixing it requires reenginering where settings are stored for Audio & Volume things.

"Raise maximum volume" currently works on a per-applet basis, just like "Manually block sleep and screen locking" in Battery & Brightness applet. So naturally KCM in system settings doesn't know anything about that state. What's worse, the keyboard volume buttons are handled by one of the applets too. I'm saying "one of", because you could add multiple instances, e.g. on different screens or panels for convenience, and only one (pretty much random) of them will be responsible for those keys, as well as effective limits.
Comment 2 Nate Graham 2022-09-13 18:45:12 UTC
Basically requires fixing Bug 440846 first.
Comment 3 Bug Janitor Service 2022-11-03 20:43:50 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-pa/-/merge_requests/148
Comment 4 Nate Graham 2022-11-03 23:54:09 UTC
Git commit 5bdcba75652a8caa3f4e3302b7277fa36c7c6417 by Nate Graham, on behalf of Bharadwaj Raju.
Committed on 03/11/2022 at 23:54.
Pushed by ngraham into branch 'master'.

kcm: make volume sliders obey "Raise maximum volume" setting
FIXED-IN: 5.27

M  +1    -3    src/kcm/package/contents/ui/VolumeControlsConfig.qml
M  +1    -1    src/kcm/package/contents/ui/VolumeSlider.qml
M  +5    -1    src/kcm/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-pa/commit/5bdcba75652a8caa3f4e3302b7277fa36c7c6417