<div dir="auto">Thanks, I'll get myself working on it.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jul 12, 2018, 08:19 Peter Hutterer <<a href="mailto:peter.hutterer@who-t.net">peter.hutterer@who-t.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jul 10, 2018 at 01:06:17PM +0530, Sagar Tewari wrote:<br>
> I would like to work on implementing right and middle click drag as<br>
> two/three finger tap followed by a single tap drag, the way it works in<br>
> synaptics driver. I found that this topic has been discussed previously:<br>
> <a href="https://lists.freedesktop.org/archives/wayland-devel/2017-September/034926.html" rel="noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/archives/wayland-devel/2017-September/034926.html</a><br>
> <br>
> But this was the single main on that topic in the archives.<br>
> <br>
> To Peter Hutterer, where can I find the patch being discussed in that mail?<br>
> Would it be better to start from scratch since the patch was apparantly<br>
> flawed?<br>
<br>
I've forwarded the attachments from that thread to you off-list.<br>
<br>
Basically, that patch was incomplete and didn't handle some states and<br>
transitions. It also wasn't documented clearly, in that it didn't explain<br>
what it *intended* to do. For hairy stuff like this, knowing what the<br>
intention was is more important than the code so we know what is a bug and<br>
what isn't.<br>
<br>
Note that the state machine is the main source of information here, the<br>
actual tapping code is just a bunch of switch statements based on the<br>
state.<br>
<br>
Cheers,<br>
   Peter<br>
</blockquote></div>