X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=src%2Fevent.js;h=cac56bd6f75bc432d1f2a5441c5ecff05892269f;hb=739644dce5312fc66a37806ff5e07c8e9784f803;hp=8d737bf4bb3e84b7e1c81c00dc4a8089da0928ec;hpb=3a9c827bf86657a0d007f66c9920d4c2b1726961;p=jquery.git diff --git a/src/event.js b/src/event.js index 8d737bf..cac56bd 100644 --- a/src/event.js +++ b/src/event.js @@ -190,9 +190,10 @@ jQuery.event = { }, // bubbling is internal - trigger: function( event, data, elem, bubbling ) { + trigger: function( event, data, elem /*, bubbling */ ) { // Event object or event type - var type = event.type || event; + var type = event.type || event, + bubbling = arguments[3]; if ( !bubbling ) { event = typeof event === "object" ? @@ -374,8 +375,13 @@ jQuery.event = { return event; }, - proxy: function( fn, proxy ) { - proxy = proxy || function() { return fn.apply( this, arguments ); }; + proxy: function( fn, proxy, thisObject ) { + if ( proxy !== undefined && !jQuery.isFunction( proxy ) ) { + thisObject = proxy; + proxy = undefined; + } + // FIXME: Should proxy be redefined to be applied with thisObject if defined? + proxy = proxy || function() { return fn.apply( thisObject !== undefined ? thisObject : this, arguments ); }; // Set the guid of unique handler to the same of original handler, so it can be removed proxy.guid = fn.guid = fn.guid || proxy.guid || this.guid++; // So proxy can be declared as an argument @@ -396,8 +402,20 @@ jQuery.event = { jQuery.event.add( this, data.live, liveHandler ); }, - teardown: function( namespaces ) { - jQuery.event.remove( this, namespaces[0], liveHandler ); + remove: function( namespaces ) { + if ( namespaces.length ) { + var remove = 0, name = new RegExp("(^|\\.)" + namespaces[0] + "(\\.|$)"); + + jQuery.each( (jQuery.data(this, "events").live || {}), function() { + if ( name.test(this.type) ) { + remove++; + } + }); + + if ( remove < 1 ) { + jQuery.event.remove( this, namespaces[0], liveHandler ); + } + } } } } @@ -509,19 +527,35 @@ jQuery.each({ }); jQuery.fn.extend({ - bind: function( type, data, fn ) { - return type === "unload" ? this.one(type, data, fn) : this.each(function() { - jQuery.event.add( this, type, fn || data, fn && data ); + bind: function( type, data, fn, thisObject ) { + if ( jQuery.isFunction( data ) ) { + if ( fn !== undefined ) { + thisObject = fn; + } + fn = data; + data = undefined; + } + fn = thisObject === undefined ? fn : jQuery.event.proxy( fn, thisObject ); + return type === "unload" ? this.one(type, data, fn, thisObject) : this.each(function() { + jQuery.event.add( this, type, fn, data ); }); }, - one: function( type, data, fn ) { - var one = jQuery.event.proxy( fn || data, function( event ) { + one: function( type, data, fn, thisObject ) { + if ( jQuery.isFunction( data ) ) { + if ( fn !== undefined ) { + thisObject = fn; + } + fn = data; + data = undefined; + } + fn = thisObject === undefined ? fn : jQuery.event.proxy( fn, thisObject ); + var one = jQuery.event.proxy( fn, function( event ) { jQuery( this ).unbind( event, one ); - return (fn || data).apply( this, arguments ); + return fn.apply( this, arguments ); }); return this.each(function() { - jQuery.event.add( this, type, one, fn && data ); + jQuery.event.add( this, type, one, data ); }); }, @@ -569,7 +603,7 @@ jQuery.fn.extend({ }, hover: function( fnOver, fnOut ) { - return this.mouseenter( fnOver ).mouseleave( fnOut ); + return this.mouseenter( fnOver ).mouseleave( fnOut || fnOver ); }, ready: function( fn ) { @@ -590,10 +624,17 @@ jQuery.fn.extend({ return this; }, - live: function( type, data, fn ) { + live: function( type, data, fn, thisObject ) { + if ( jQuery.isFunction( data ) ) { + if ( fn !== undefined ) { + thisObject = fn; + } + fn = data; + data = undefined; + } jQuery( this.context ).bind( liveConvert( type, this.selector ), { - data: fn && data, selector: this.selector, live: type - }, fn || data ); + data: data, selector: this.selector, live: type + }, fn, thisObject ); return this; }, @@ -621,7 +662,7 @@ function liveHandler( event ) { jQuery.each(elems, function() { event.currentTarget = this.elem; - event.data = this.fn.data + event.data = this.fn.data; if ( this.fn.apply( this.elem, args ) === false ) { return (stop = false); } @@ -718,7 +759,7 @@ jQuery.each( ("blur,focus,load,resize,scroll,unload,click,dblclick," + // Handle event binding jQuery.fn[ name ] = function( fn ) { - return fn ? this.bind (name, fn ) : this.trigger( name ); + return fn ? this.bind( name, fn ) : this.trigger( name ); }; });