search

jyoungblood / smoke.js

framework-agnostic styled alert system for javascript
http://smoke.js.org/
930 stars 107 forks source link

Touch events cause problems with smoke.confirm on mobile Safari #29

Open homp opened 12 years ago

homp commented 12 years ago

Really loving this library, really nice.

But - there seems to be an issue with touch functions (touchstart, touchend, etc) in mobile safari.

Check the modified demo.html I've put up at http://mima.x.se/nils/smoke/demo.html

Apart from demo.html I've used the original package from github.

The confirm link has been modified to activate on "touchend" which screws up the confirm (it disappears within a fraction of a second). Binding it to a click event works fine.

User String on the iPad used:

Mozilla/5.0 (iPad; CPU OS 5_1 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9B176 Safari/7534.48.3

homp commented 12 years ago

Tracked this to the "close on background click" functionality. Removed that and it works fine.