only override behaviour if we really have a click delay - not just every browser supporting 'ontouchstart'

[jamuhl]

I run into an issue testing our site with casperjs (phantomjs). When i added backbone.touch the tests failed.

I changed line 32 to:

isTouch : 'ontouchstart' in this.document && /iPhone|iPod|iPad|Android|BlackBerry|PlayBook/.test(navigator.userAgent),

like this only mobile devices and tablets get the click events changed and all tests green again.

[nervetattoo]

In which ways does it fail? Does phantomjs come with touch* handlers?
I guess that kind of fix solves it mostly, but I would prefer to not use device detection as thats a list that needs to be maintained in the future.

[jamuhl]

Yes. Phantomjs implements some of the touch events. Another option would be to explicit look if it is phantomjs, like here:

https://github.com/cheeaun/tappable/pull/11/files

But actually i'm not sure if it's a good idea - feels wrong.

[nervetattoo]

Since it seems like this will be corrected in phantomjs at some point (given the attitude of the maintainer) I think doing specific sniffing of phantomjs for now is my preferred solution. Feel free to do a pull request, if not I will add it later.

[jamuhl]

added a PR.