Skip to content

kcms/touchpad: Make QML more declarative

Jakob Petsovits requested to merge work/jpetso/refactor-touchpad-kcm into master

Commit messages

Starting with tip of tree:

kcms/touchpad: Make QML more declarative

Mostly a rip-off of kcm_mouse commit 9c60de34 (MR !2082 (merged)), with some later polish included from the start. Reduces the diff between both KCMs.

A lot of that imperative QML logic can be replaced with bindings, which simplifies the code quite a bit. The "Reset" button is now also properly disabled once settings are reverted to the last saved state.

The C++ changes primarily ensure that change notification signals are emitted when the value changes.

Bonus commit: kcms/mouse: Tweak "enabled" checkbox state

From the parallel kcm_touchpad commit, to keep both in sync.

Testing

Open the "Touchpad" KCM in System Settings, make sure options still work on Wayland as well as X11.

Possible reviewers/testers

@ratijas, @nicolasfella, @redstrate, @ngraham, @duha

Edited by Jakob Petsovits

Merge request reports

Loading