D29365: Increase values for touchpad acceleration that can be set from kcm libinput UI

[Giusy Margarita]

kurmikon updated this revision to Diff 82966.
kurmikon added a comment.


  Diff updated to a better version. So, I'm explaining what happens here.
  
  A RowLayout is placed where the old slider was. This contains a new slider and an additional spinbox.
  
  The spinbox gets float values by the user as reported by libinput documentation: [-1, 1]. Displayed value is converted in the internal integer value [-100, 100] with a stepsize of 1.
  
  The slider is basically the same as before, range [1, 11], only values multiple of 0,2 can be selected. An additional property AccelSpeedValue is set to convert the internal value to the format used by the spinbox.
  
  What happens when one control is modified. If the spinbox is modified, its new internal value is passed to //onAccelSpeedChanged// method of the RowLayout. Il the slider is modified, it's AccelSpeedValue is passed to //onAccelSpeedChanged// method of the RowLayout.
  
  What //onAccelSpeedChanged// does. It takes the arguments and compare it three times.
  
  1. with the slider accelSpeedValue; if it's different, the slider has been updated
  2. with the spinbox internal value; if it's different, the spinbox has been updated
  3. the argument divided by 100 with the libinput pointer acceleration; if it's different, the libinput property is updated and root.changeSignal() is invoked.
  
  Surely the method is called another time because of the update of the other control, but all checks will result false and nothing happens, avoiding loops, double writings and other issues.
  
  Compiled and tested on my system, it's working. Please, test on yours. Can't do better than that with my poor knowledge.

REPOSITORY
  R119 Plasma Desktop

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D29365?vs=82323&id=82966

REVISION DETAIL
  https://phabricator.kde.org/D29365

AFFECTED FILES
  kcms/touchpad/kcm/libinput/touchpad.qml

To: kurmikon, #vdg, #plasma
Cc: ndavis, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200515/95344f77/attachment-0001.htm>