Depends on D28310
FEATURE: 403843
FIXED-IN: 5.19.0
Details
Details
- Reviewers
davidedmundson ervin bport crossi hchain - Group Reviewers
KWin Plasma - Commits
- R119:0cf73170479b: KCM/mouse KCM/touchpad: Add a Scroll speed setting for wayland
Diff Detail
Diff Detail
- Repository
- R119 Plasma Desktop
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 24316 Build 24334: arc lint + arc unit
| kcms/touchpad/src/kcm/libinput/touchpad.qml | ||
|---|---|---|
| 607 ↗ | (On Diff #78619) | We can make it slower with a number between 0 and 1. |
| kcms/touchpad/src/kcm/libinput/touchpad.qml | ||
|---|---|---|
| 607 ↗ | (On Diff #78619) | I suspect that a scroll factor less than 1 may screw up discrete axis values. Not sure whether it might be a problem, though. |
| kcms/touchpad/src/kcm/libinput/touchpad.qml | ||
|---|---|---|
| 607 ↗ | (On Diff #78619) | My testing went ok with a 0.1 scrollFactor, although not extensive (scroll in XWayland Firefox and in qtcreator Wayland mostly). |
Comment Actions
Use onMoved, allow to slow down the scroll speed, use presset values for the scroll speed slider
Comment Actions
These KCMs have X11 versions too. They are not Wayland-specific. I'm not a fan of adding this feature only on Wayland.
Comment Actions
By those KCM, I meant their frontend (they have each 1 for X and 1 for Wayland).
mouse kcm for X is in main_deviceless.qml
touchpad kcm for X is in touchpadconfigxlib.cpp
And this feature can only be implemented for Wayland.
| kcms/mouse/kcm/libinput/main.qml | ||
|---|---|---|
| 333 ↗ | (On Diff #78619) | nitpick: I'd put the closing square bracket on the next line and you can drop the semicolumn |
| 343 ↗ | (On Diff #78619) | You can drop the semicolumn in there |
| 346 ↗ | (On Diff #78619) | ditto |
| kcms/touchpad/src/kcm/libinput/touchpad.qml | ||
| 631 ↗ | (On Diff #78619) | Same comment applies than for mouse |
| 638 ↗ | (On Diff #78619) | ditto |
| 644 ↗ | (On Diff #78619) | ditto |