Skip to content
Permalink
Browse files

jquery event: closes #4033. e.currentTarget wasn't being enforced for…

… native events (IE doesn't have it)
  • Loading branch information
flesler committed Feb 17, 2009
1 parent 72cf42b commit 0a1b0db347119a2e45d3d31285223ead27b64506
Showing with 16 additions and 1 deletion.
  1. +2 −1 src/event.js
  2. +14 −0 test/unit/event.js
@@ -240,7 +240,8 @@ jQuery.event = {
var all, handlers;

event = arguments[0] = jQuery.event.fix( event || window.event );

event.currentTarget = this;

// Namespaced event handlers
var namespaces = event.type.split(".");
event.type = namespaces.shift();
@@ -417,6 +417,20 @@ test("trigger(eventObject, [data], [fn])", function() {
$parent.unbind().remove();
});

test("jQuery.Event.currentTarget", function(){
expect(2);

var counter = 0,
$elem = jQuery('<button>a</button>').click(function(e){
equals( e.currentTarget, this, "Check currentTarget on "+(counter++?"native":"fake") +" event" );
});

// Fake event
$elem.trigger('click');
// Native event (#4033)
triggerEvent( $elem[0], 'click' );
});

test("toggle(Function, Function, ...)", function() {
expect(11);

0 comments on commit 0a1b0db

Please sign in to comment.
You can’t perform that action at this time.