Bug 459639

Summary:	Nothing happens when I scroll on top of the sliders in Audio KCM
Product:	[Applications] systemsettings	Reporter:	Patrick Silva <bugseforuns>
Component:	kcm_pulseaudio	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	minor	CC:	me, nate, nowrep
Priority:	NOR
Version:	5.27.80
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-pa/-/commit/ec336f6451e92b8946eb750068ebe31875a2fd4e	Version Fixed In:	6.0
Sentry Crash Report:

Description Patrick Silva 2022-09-25 10:31:01 UTC

SUUMARY
We can adjust the volume level by scrolling on top of the sliders of plasma-pa applet.
But we cannot do the same with the sliders in Audio KCM.

STEPS TO REPRODUCE
1. open Audio KCM
2. hover over any slider
3. scroll up/down

OBSERVED RESULT
nothing happens

EXPECTED RESULT
volume level changes

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.26.80
KDE Frameworks Version: 5.99.0
Qt Version: 5.15.6
Graphics Platform: Wayland

Comment 1 Nate Graham 2022-09-26 19:25:28 UTC

Can reproduce. Seems like an issue specific to the custom slider object in this KCM.

Comment 2 Bug Janitor Service 2024-01-24 04:08:50 UTC

A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-pa/-/merge_requests/234

Comment 3 fanzhuyifan 2024-01-24 18:02:34 UTC

Git commit 661f40d7a4f81fd0aebf67c09a9dd5ea17b6fb1f by Yifan Zhu.
Committed on 24/01/2024 at 19:00.
Pushed by fanzhuyifan into branch 'master'.

kcm/VolumeSlider: fix scroll

stepSize was not set to disable tickmarks. However scrolling also
depends on stepSize. qqc2-desktop-stlye implements scrolling via
increase()/descrease() functions, which default to steps of 0.1
when stepSize is not set.

Override them to get scrolling working without setting stepSize.
Related: bug 409325
FIXED-IN: 6.0

M  +6    -2    src/kcm/ui/VolumeSlider.qml

https://invent.kde.org/plasma/plasma-pa/-/commit/661f40d7a4f81fd0aebf67c09a9dd5ea17b6fb1f

Comment 4 fanzhuyifan 2024-01-24 18:16:07 UTC

Git commit ec336f6451e92b8946eb750068ebe31875a2fd4e by Yifan Zhu.
Committed on 24/01/2024 at 19:14.
Pushed by fanzhuyifan into branch 'Plasma/6.0'.

kcm/VolumeSlider: fix scroll

stepSize was not set to disable tickmarks. However scrolling also
depends on stepSize. qqc2-desktop-stlye implements scrolling via
increase()/descrease() functions, which default to steps of 0.1
when stepSize is not set.

Override them to get scrolling working without setting stepSize.
Related: bug 409325
FIXED-IN: 6.0


(cherry picked from commit 661f40d7a4f81fd0aebf67c09a9dd5ea17b6fb1f)

M  +6    -2    src/kcm/ui/VolumeSlider.qml

https://invent.kde.org/plasma/plasma-pa/-/commit/ec336f6451e92b8946eb750068ebe31875a2fd4e