<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>