Bug 403843 - Libinput touchpad KCM needs a scroll speed setting
Summary: Libinput touchpad KCM needs a scroll speed setting
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_touchpad (show other bugs)
Version: 5.17.5
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
: 401954 (view as bug list)
Depends on:
Blocks:
 
Reported: 2019-02-01 19:11 UTC by Nate Graham
Modified: 2020-10-24 16:10 UTC (History)
9 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.19.0 (Wayland only)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2019-02-01 19:11:04 UTC
With Libinput, there's no  concept of "scroll by n lines" the way there was with evdev. Instead Libinput leaves it up to the DE to take the Libinput-provided scroll angle/delta information and scale that appropriately before passing it to the UI toolkit for toolkit-specific scroll handling.

We should support this, since right now there is no UI to configure touchpad scroll sensitivity/speed.

For more information, see https://gitlab.freedesktop.org/libinput/libinput/issues/185
Comment 1 Nate Graham 2019-12-14 16:14:02 UTC
There's a WIP Libinput driver patch that would expose the API that we could hook into from the touchpad KCM: https://gitlab.freedesktop.org/xorg/driver/xf86-input-libinput/merge_requests/12
Comment 2 Tim Richardson 2020-01-22 21:45:17 UTC
I have libinput 1.15 on my Neon laptop, and I can see the new UI, and it works. Thanks.
Comment 3 Méven Car 2020-03-26 09:47:05 UTC
Well Tim Richardson, I wonder how you can have a UI for this when we have no implementation for it and upstream has not landed their patch yet.
I guess You meant something else than "scroll by n lines" for touchpad that we talk about here.
Comment 4 Tim Richardson 2020-03-26 10:23:51 UTC
I interpreted this bug as 'no UI to configure touchpad scroll sensitivity/speed'. At least, this was the interpretation that I responded to. Such as UI was missing, and then it was added, hence my reference to the new UI, which was actually what I wanted all along. Maybe Nate's initial comment does not make sense, he goes from "scroll by n lines" to speed/sensitivity. They are different things. Scroll by n lines is mouse wheel to me.
Comment 5 Tim Richardson 2020-03-26 10:36:45 UTC
Although changing the pointer speed in the touchpad kcm does not actually seem to change the speed of two finger scrolling, which I obviously don't use very much. 
So in fact I actually confirm that there is a touchpad kcm when using libinput, and pointer sensitivity can be controlled, but there is no control over  scrolling sensitivity.
Comment 6 Claudius Ellsel 2020-03-26 18:15:10 UTC
(In reply to Tim Richardson from comment #5)
> Although changing the pointer speed in the touchpad kcm does not actually
> seem to change the speed of two finger scrolling, which I obviously don't
> use very much. 
> So in fact I actually confirm that there is a touchpad kcm when using
> libinput, and pointer sensitivity can be controlled, but there is no control
> over  scrolling sensitivity.

So basically you mixed up a setting for pointer sensitivity with scroll sensitivity?

Also, what do you mean with "new UI"?
Comment 7 Tim Richardson 2020-03-26 19:50:43 UTC
"New UI": libinput exposes a number of configurations for touchpads. For a long time, most of these options were not exposed to the touchpad kcm. This changed in Jan 2020 when a much richer kcm arrived, hence "new UI".
Comment 8 Claudius Ellsel 2020-03-26 21:09:38 UTC
Ah, I wasn't aware of that. Thanks for mentioning!
Comment 9 Méven Car 2020-04-07 17:08:56 UTC
Git commit 564c739371c8076728c264154847316c64e4c1ec by Méven Car.
Committed on 07/04/2020 at 17:08.
Pushed by meven into branch 'master'.

Wayland: Allow to set a scroll Factor for input devices

Summary:
Mouse and touchpad wheel events are concerned.

KCM patch: D28331

Test Plan: build, ctest, manual set of scrollfactor value via qdbusviewer

Reviewers: #kwin, davidedmundson, zzag, bport, ervin, apol

Reviewed By: #kwin, davidedmundson, ervin, apol

Subscribers: apol, kwin

Tags: #kwin

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

M  +24   -0    autotests/libinput/device_test.cpp
M  +23   -2    libinput/device.cpp
M  +12   -0    libinput/device.h
M  +2    -2    libinput/events.cpp

https://commits.kde.org/kwin/564c739371c8076728c264154847316c64e4c1ec
Comment 10 Méven Car 2020-04-07 17:09:52 UTC
Git commit 0cf73170479bb271fb7498e67d38d3c6255594a4 by Méven Car.
Committed on 07/04/2020 at 17:09.
Pushed by meven into branch 'master'.

KCM/mouse KCM/touchpad: Add a Scroll speed setting for wayland

Summary:
Depends on D28310
FIXED-IN: 5.19.0

Test Plan: {F8201345}

Reviewers: #kwin, #plasma, davidedmundson, ervin, bport, crossi, hchain

Reviewed By: ervin

Subscribers: ngraham, zzag, plasma-devel

Tags: #plasma

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

M  +7    -1    kcms/mouse/backends/kwin_wl/kwin_wl_device.cpp
M  +11   -0    kcms/mouse/backends/kwin_wl/kwin_wl_device.h
M  +63   -0    kcms/mouse/kcm/libinput/main.qml
M  +5    -0    kcms/touchpad/src/backends/kwin_wayland/kwinwaylandtouchpad.cpp
M  +18   -0    kcms/touchpad/src/backends/kwin_wayland/kwinwaylandtouchpad.h
M  +63   -0    kcms/touchpad/src/kcm/libinput/touchpad.qml

https://commits.kde.org/plasma-desktop/0cf73170479bb271fb7498e67d38d3c6255594a4
Comment 11 Nate Graham 2020-06-15 16:01:52 UTC
*** Bug 401954 has been marked as a duplicate of this bug. ***