Skip to content

Commit

Permalink
Fixed and improved jQuery's event system: The target property of the …
Browse files Browse the repository at this point in the history
…event object is now fixed (IE and Safari), bind() accepts now a third parameter "amount" to run the function only the specified number of times, oneclick etc. uses this new bind - it removes the handler when it is not necessary anymore, therefore providing better performance, see accordion for test/demo ( http://joern.jquery.com/accordion/accordion.html )
  • Loading branch information
jzaefferer committed Oct 26, 2006
1 parent 5131224 commit 897fbfe
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 21 deletions.
20 changes: 4 additions & 16 deletions src/event/event.js
Expand Up @@ -1583,29 +1583,17 @@ new function(){
var o = e[i];

// Handle event binding
jQuery.fn[o] = function(f){
return f ? this.bind(o, f) : this.trigger(o);
jQuery.fn[o] = function(f, amount){
return f ? this.bind(o, f, amount) : this.trigger(o);
};

// Handle event unbinding
jQuery.fn["un"+o] = function(f){ return this.unbind(o, f); };

// Finally, handle events that only fire once
jQuery.fn["one"+o] = function(f){
// Attach the event listener
return this.each(function(){

var count = 0;

// Add the event
jQuery.event.add( this, o, function(e){
// If this function has already been executed, stop
if ( count++ ) return true;

// And execute the bound function
return f.apply(this, [e]);
});
});
// use bind with amount param to bind only once
return this.bind(o, f, 1);
};

};
Expand Down
27 changes: 22 additions & 5 deletions src/jquery/jquery.js
Expand Up @@ -2271,7 +2271,7 @@ jQuery.extend({
handle: function(event) {
if ( typeof jQuery == "undefined" ) return false;

event = event || jQuery.event.fix( window.event );
event = jQuery.event.fix( event );

// If no correct event was found, fail
if ( !event ) return false;
Expand All @@ -2295,16 +2295,19 @@ jQuery.extend({
},

fix: function(event) {
if ( event ) {
if(jQuery.browser.msie) {
event = window.event;
event.preventDefault = function() {
this.returnValue = false;
};

event.stopPropagation = function() {
this.cancelBubble = true;
};
event.target = event.srcElement;
} else if(jQuery.browser.safari && event.target.nodeType == 3) {
event = jQuery.extend({}, event);
event.target = event.target.parentNode;
}

return event;
}

Expand Down Expand Up @@ -3396,15 +3399,29 @@ jQuery.macros = {
* } )
* @desc Stop only an event from bubbling by using the stopPropagation method.
*
* @example $("form").bind( "submit", function(event) {
* // do something after submit
* }, 1 )
* @desc Executes the function only on the first submit event and removes it afterwards
*
* @name bind
* @type jQuery
* @param String type An event type
* @param Function fn A function to bind to the event on each of the set of matched elements
* @param Number amount An optional amount of times to execute the bound function
* @cat Events
*/
bind: function( type, fn ) {
bind: function( type, fn, amount ) {
if ( fn.constructor == String )
fn = new Function("e", ( !fn.indexOf(".") ? "jQuery(this)" : "return " ) + fn);
if( amount > 0 ) {
var element = this, handler = fn, count = 0;
fn = function(e) {
if( ++count >= amount )
jQuery(element).unbind(type, fn);
handler.apply(element, [e]);
};
}
jQuery.event.add( this, type, fn );
},

Expand Down

0 comments on commit 897fbfe

Please sign in to comment.