deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

jquery event: Fixing event.currentTarget for live().

Browse source
This commit is contained in:
Ariel Flesler 2009-02-23 13:27:48 +00:00
parent 883450b702
commit 48164ee603
2 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
src/event.js
View file

@ -577,6 +577,7 @@ function liveHandler( event ){
}); });
jQuery.each(elems, function(){ jQuery.each(elems, function(){
event.currentTarget = this.elem;
if ( this.fn.call(this.elem, event, this.fn.data) === false ) if ( this.fn.call(this.elem, event, this.fn.data) === false )
return (stop = false); return (stop = false);
}); });

13
test/unit/event.js
View file

@ -489,7 +489,7 @@ test("toggle(Function, Function, ...)", function() {
}); });
test(".live()/.die()", function() { test(".live()/.die()", function() {
expect(46); expect(49);
var submit = 0, div = 0, livea = 0, liveb = 0; var submit = 0, div = 0, livea = 0, liveb = 0;
@ -649,6 +649,17 @@ test(".live()/.die()", function() {
// Cleanup // Cleanup
jQuery("span#liveSpan1 a, span#liveSpan1, span#liveSpan2 a, span#liveSpan2").die("click"); jQuery("span#liveSpan1 a, span#liveSpan1, span#liveSpan2 a, span#liveSpan2").die("click");
// Test this, target and currentTarget are correct
jQuery('span#liveSpan1').live('click', function(e){
equals( this.id, 'liveSpan1', 'Check the this within a live handler' );
equals( e.currentTarget.id, 'liveSpan1', 'Check the event.currentTarget within a live handler' );
equals( e.target.nodeName.toUpperCase(), 'A', 'Check the event.target within a live handler' );
});
jQuery('span#liveSpan1 a').click();
jQuery('span#liveSpan1').die('click');
}); });
/* /*