Skip to content

Commit

Permalink
Fix defaultPrevented tests for Android 2.3
Browse files Browse the repository at this point in the history
This property is not present in Android 2.3 so it shouldn't be used in
tests. Instead, use `isDefaultPrevented()` which is guaranteed to be
cross-platform.
  • Loading branch information
mislav committed Dec 1, 2013
1 parent 627801c commit 17b0757
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions test/zepto.html
Original file line number Diff line number Diff line change
Expand Up @@ -2497,7 +2497,7 @@ <h1>Zepto DOM unit tests</h1>
return false
})
.bind('click', function (e) {
prevented = e.defaultPrevented
prevented = e.isDefaultPrevented()
})
.trigger('click')

Expand Down Expand Up @@ -2560,21 +2560,18 @@ <h1>Zepto DOM unit tests</h1>
testTriggerEventCancelled: function(t){
var event = $.Event('custom'),
element = $('<div/>'),
defaultPrevented = false,
isDefaultPrevented = false

t.refute(event.isDefaultPrevented())

element.bind('custom', function(e){
e.preventDefault()
defaultPrevented = e.defaultPrevented
isDefaultPrevented = e.isDefaultPrevented()
})

element.trigger(event)

t.assertTrue(event.isDefaultPrevented())
t.assertTrue(defaultPrevented)
t.assertTrue(isDefaultPrevented)
},

Expand Down

0 comments on commit 17b0757

Please sign in to comment.