<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - Double tap not always recognized"
href="https://bugs.freedesktop.org/show_bug.cgi?id=90172#c13">Comment # 13</a>
on <a class="bz_bug_link
bz_status_ASSIGNED "
title="ASSIGNED - Double tap not always recognized"
href="https://bugs.freedesktop.org/show_bug.cgi?id=90172">bug 90172</a>
from <span class="vcard"><a class="email" href="mailto:lisec.velimir@gmail.com" title="Velimir Lisec <lisec.velimir@gmail.com>"> <span class="fn">Velimir Lisec</span></a>
</span></b>
<pre>I've been playing with this some more and it turns out your patch works. The
problem was I didn't clean git repositories before compiling. Because of that
every time I started weston it started from the very first folder I compiled
to, and that one didn't have patch applied. I always started event-debug from
the right folder and that's why it reported two button presses but client only
registered one.
Also, DEFAULT_TAP_TIMEOUT_PERIOD should be reduced to 150 ms (or maybe 140?).
When I was using gedit some double taps didn't get registered. That happened
when I tapped slower so I guess those taps exceeded the time required by the
application. That happened with DEFAULT_TAP_TIMEOUT_PERIOD set to 180. When I
reduced it to 150 it always worked. And with smaller timeout period there's
also smaller delay between lifting finger for the second time and that getting
registered as a second tap. With timeout period set to 150 I could still
reliably do tap and drag. I haven't tried anything smaller than 150, but i
think it would work with 140. I'll try that tomorrow.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>