https://bugs.kde.org/show_bug.cgi?id=359460
--- Comment #15 from Rajeesh K V <rajeeshknamb...@gmail.com> --- (In reply to Mykola Krachkovsky from comment #14) > (In reply to Rajeesh K V from comment #13) > > (In reply to Mykola Krachkovsky from comment #11) > > > > > In file kcms/touchpad/src/backends/x11/synapticstouchpad.h class > > > SynapticsTouchpad redefines (line 34) fields: > > > QStringList m_scaleByResX, m_scaleByResY, m_toRadians; > > > from parent class XlibTouchpad (file > > > kcms/touchpad/src/backends/x11/xlibtouchpad.h, line 79). > > > So usage that fields in files synapticstouchpad.cpp (adding data) and > > > xlibtouchpad.cpp (reading data) works with different variables. And > > > XlibTouchpad::getPropertyScale returns default 1.0 instead of correct > > > value. > > > > Indeed, that would explain the problem. There was a similar bug but that was > > fixed. Can you raise a review request for this patch or shall I do it for > > you? > > > > I don't know what I should do. Okay, I'll take care of it. > > > > (In reply to Mykola Krachkovsky from comment #11) > > > But KCM works not ok > > What do you mean by KCM doesn't work? > > It looks like VertScrollDelta and HorizScrollDelta are written to > ~/.config/touchpadrc but aren't actually set to device. But MaxTapMove, > CircScrollDelta works "ok". > There is also "Active settings don't match saved settings. You currently see > saved settings." message when I press "Show active settings" I got 10mm for > vertical and 0.1mm for horizontal. This requires my patch to work as well. Scroll delta maximum is hard coded to 10 in the kcfg file. I'll fold both into single patch and upstream. -- You are receiving this mail because: You are watching all bug changes.
