Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fixed yet another duplicate tap event bug. #1381

Closed
wants to merge 1 commit into from

1 participant

@martinkou

Line 81 of jquery.mobile.event.js says...

            function clearTapHandlers() {
                touching = false;
                clearTimeout(timer);
                $(this).unbind("vclick", clickHandler).unbind("vmousecancel", clearTapHandlers);
            }

The unbind line doesn't actually work because the this object at the moment clearTapHandlers() is called is not the DOM element with the tap event.

@martinkou martinkou closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 6, 2011
  1. Fixed yet another duplicate tap event bug.

    Martin Kou authored
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  js/jquery.mobile.event.js
View
2  js/jquery.mobile.event.js
@@ -81,7 +81,7 @@ $.event.special.tap = {
function clearTapHandlers() {
touching = false;
clearTimeout(timer);
- $(this).unbind("vclick", clickHandler).unbind("vmousecancel", clearTapHandlers);
+ $this.unbind("vclick", clickHandler).unbind("vmousecancel", clearTapHandlers);
}
function clickHandler(event) {
Something went wrong with that request. Please try again.