Review Request 127306: Touchpad KCM: Fix synaptics driver issues

[Rajeesh K V]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127306/
-----------------------------------------------------------

(Updated March 9, 2016, 7:31 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma, David Edmundson and Xuetian Weng.


Changes
-------

Submitted with commit 86b59198f1c60236034567dc804b578376f8d06c by Rajeesh K V to branch Plasma/5.6.


Bugs: 359460
    https://bugs.kde.org/show_bug.cgi?id=359460


Repository: plasma-desktop


Description
-------

Fixes few things:
1.  Remove spurious members of SynapticsTouchpad class from parent class XlibTouchpad, fixing wrong scroll delta values.
2.  Enlarge vertical,horizontal&circular scroll delta and maxtapmove allowing value more than 100. The max value supported by driver is 1000.

I would like to get this fix in 5.6 final.


Diffs
-----

  kcms/touchpad/src/backends/x11/synapticstouchpad.h de75125 
  kcms/touchpad/src/backends/x11/synapticstouchpad.cpp 1e01894 
  kcms/touchpad/src/backends/x11/xlibtouchpad.h 0eb58fb 
  kcms/touchpad/src/backends/x11/xlibtouchpad.cpp 441bcad 
  kcms/touchpad/src/kcm/touchpad.kcfg d4d127c 
  kcms/touchpad/src/kcm/ui/scroll.ui d1c2f76 
  kcms/touchpad/src/kcm/ui/tap.ui 389db5e 

Diff: https://git.reviewboard.kde.org/r/127306/diff/


Testing
-------

Built the RPM, compile tested and locally tested. Bug reporters confirm patch fixes the issue.


Thanks,

Rajeesh K V

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160309/da3cbdf0/attachment.html>