Eating clicks on Windows > 7 about fastclick

[mattcg]

I've tested the version tagged at 3.0.0 with Chrome running on Windows 7 in an emulator, with 'emulate touch events' enabled in the web inspector settings. It seems fine now, though this might be a false positive. Waiting for confirmation from the reporter.

[RByers]

Note that we made the decision NOT to delay click events on Chrome desktop (i.e. we don't support double-tap to zoom) precisely so that work-arounds like this library aren't necessary. Was the fix here to just disable support entirely for Chrome desktop UserAgents? That would be ideal. Eg. WebKit does a lot of work to try to generate good click events (eg. adjusting the position of the touch to the most likely element under the finger), so you don't want to intefere with that if you can help it. Unfortunately this is still necessary on Chrome for Android when the page is zoomable, but if I had my way we'd get rid of the delay there too :-)

[mattcg]

Note that we made the decision NOT to delay click events on Chrome desktop

That's excellent news and thank you for posting this update.

We opted for a more a generic fix instead of checking the user agent. Before the fix, FastClick behaved like this:

• if window.ontouchstart != undefined, add workaround listeners
• if a click event is fired without a preceding touchend event, cancel the click

Of course, the unintended effect was that on platforms like Chrome/Windows 7 pages were made unclickable. So now we do this:

• if window.ontouchstart != undefined, add workaround listeners
• if a click event is fired without a preceding touchend event, let the click through

This still isn't ideal because we're still adding the workaround listeners even if they're not needed by the device.

[mattcg]

WebKit does a lot of work to try to generate good click events (eg. adjusting the position of the touch to the most likely element under the finger), so you don't want to intefere with that if you can help it.

I'm with you on this: I can't wait for the day when FastClick won't be needed at all.