<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/123621/">https://git.reviewboard.kde.org/r/123621/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 4th, 2015, 6:40 p.m. UTC, <b>Rajeesh K Nambiar</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Reverse scrolling works in KF5 based apps (tested with dolphin, konsole etc) and gtk+ apps like firefox here - using synaptics driver. Also see the discussion in bug report https://bugs.kde.org/show_bug.cgi?id=342929 where only KDE4 apps are reported to be affected. Could you share more details?</p></pre>
</blockquote>
<p>On May 5th, 2015, 2:09 a.m. UTC, <b>Yue Liu</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">reverse scroll of touchpad and reverse scroll of mouse/trackpoint are different things since kcm_touchpad and kcm_input (mouse) works separately. I don't think synaptics driver support mouse. For that bug report I guess some commenters didn't notice it is for touchpad kcm</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">had a look at kcm_touchpad code and I feel my patch my break touchpad if scroll direction first reverted by xinput then reverted again by synaptics. I don't no how does xinput works together with touchpad driver and I don't have a touchpad for testing.</p></pre>
<br />
<p>- Yue</p>
<br />
<p>On May 4th, 2015, 9:10 a.m. UTC, Yue Liu wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Yue Liu.</div>
<p style="color: grey;"><i>Updated May 4, 2015, 9:10 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-desktop
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The old swap-button-map method for reverse scroll doesn't work for Qt5, this patch uses XInput2 api to negate "Evdev Scroll Distance" for device with real scroll wheel, and swap "Evdev Wheel Emulation Axes" when device uses emulated scroll wheel, such as Thinkpad Trackpoint.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">built on archlinux and tested by replacing kcm_input.so, new kcm works on my machine.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kcms/input/CMakeLists.txt <span style="color: grey">(6cbe207)</span></li>
<li>kcms/input/mouse.cpp <span style="color: grey">(1450de8)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/123621/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>