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

jquery event: closes #4033. e.currentTarget wasn't being enforced for native events (IE doesn't have it)

Browse source
This commit is contained in:
Ariel Flesler 2009-02-17 12:38:16 +00:00
parent 72cf42bcbd
commit 0a1b0db347
2 changed files with 16 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/event.js
View file

@ -240,7 +240,8 @@ jQuery.event = {
var all, handlers; var all, handlers;
event = arguments[0] = jQuery.event.fix( event || window.event ); event = arguments[0] = jQuery.event.fix( event || window.event );
event.currentTarget = this;
// Namespaced event handlers // Namespaced event handlers
var namespaces = event.type.split("."); var namespaces = event.type.split(".");
event.type = namespaces.shift(); event.type = namespaces.shift();

14
test/unit/event.js
View file

@ -417,6 +417,20 @@ test("trigger(eventObject, [data], [fn])", function() {
$parent.unbind().remove(); $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() { test("toggle(Function, Function, ...)", function() {
expect(11); expect(11);