<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <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#c15">Comment # 15</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 <a class="email" href="mailto:lisec.velimir@gmail.com" title="Velimir Lisec <lisec.velimir@gmail.com>"> Velimir Lisec</a> <pre>(In reply to Peter Hutterer from <a href="show_bug.cgi?id=90172#c14">comment #14</a>) > I don't think we can actually reduce the first tap timeout by much. If you > hit a ~200ms delay on a "normal tap" (<a href="show_bug.cgi?id=90172#c9">comment #9</a>) then reducing it further > risks missing it altogether. Yes, but if I'm not mistaken in the current implementation timer is reset after the first release event arrives. That means that delay between first touch down event and first release event is DEFAULT_TAP_TIMEOUT_PERIOD, and delay between first release event and second touch event is also DEFAULT_TAP_TIMEOUT_PERIOD. 200 ms from "normal tap" is delay between first and second touch down event. So with current implementation, if DEFAULT_TAP_TIMEOUT_PERIOD is set at 180, delay between first and second touch down events can theoretically go up to 360 ms (180 + 180) and the tap would still be recognized. > Here's a patch that reduces the second tap timeout, I think that's better to > rely on. That should then also reduce the total time so we're more likely to > be interpreted as a click in the default settings of the client. I get the best results with the first patch applied and DEFAULT_TAP_TIMEOUT_PERIOD set to 150 ms so I think you should use that one. Also, DEFAULT_TAP_TIMEOUT_PERIOD is used as a timeout between finger lifted during drag and finger lowered to continue drag. That time should be increased, and I think 180 ms is to low, 300 (or 350?) should be good value for that timeout (that's a guess, i haven't tried it. Under synaptics driver I use a timeout of 700 ms, but that is to long as a default. Ideally, that value should be configurable).</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>