Modified onexxx handlers to unbind themselve when executed; Reintroduced event fixes...
[jquery.git] / src / event / event.js
index 3610122..7a61773 100644 (file)
@@ -80,7 +80,7 @@ jQuery.fn.extend({
                        var p = (e.type == "mouseover" ? e.fromElement : e.toElement) || e.relatedTarget;
        
                        // Traverse up the tree
-                       while ( p && p != this ) p = p.parentNode;
+                       while ( p && p != this ) try { p = p.parentNode } catch(e) { p = this; };
                        
                        // If we actually just moused on to a sub-element, ignore it
                        if ( p == this ) return false;
@@ -107,6 +107,8 @@ jQuery.fn.extend({
         * Please ensure you have no code in your <body> onload event handler, 
         * otherwise $(document).ready() may not fire.
         *
+        * You can have as many $(document).ready events on your page as you like.
+        *
         * @example $(document).ready(function(){ Your code here... });
         *
         * @name ready
@@ -153,6 +155,9 @@ jQuery.extend({
                                // Reset the list of functions
                                jQuery.readyList = null;
                        }
+                       // Remove event lisenter to avoid memory leak
+                       if ( jQuery.browser.mozilla || jQuery.browser.opera )
+                               document.removeEventListener( "DOMContentLoaded", jQuery.ready, false );
                }
        }
 });
@@ -720,11 +725,15 @@ new function(){
                 * Trigger the load event of each matched element. This causes all of the functions
                 * that have been bound to thet load event to be executed.
                 *
+                * Marked as private: Calling load() without arguments throws exception because the ajax load
+                * does not handle it.
+                *
                 * @example $("p").load();
                 * @before <p onload="alert('Hello');">Hello</p>
                 * @result alert('Hello');
                 *
                 * @name load
+                * @private
                 * @type jQuery
                 * @cat Events/Browser
                 */
@@ -1583,20 +1592,15 @@ new function(){
                
                // 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;
-                               
-                                       // And execute the bound function
-                                       return f.apply(this, [e]);
-                               });
-                       });
+                       // save cloned reference to this
+                       var element = jQuery(this);
+                       var handler = function() {
+                               // unbind itself when executed
+                               element.unbind(o, handler);
+                               // apply original handler with the same arguments
+                               f.apply(this, arguments);
+                       };
+                       return this.bind(o, handler);
                };
                        
        };
@@ -1649,10 +1653,10 @@ new function(){
 };
 
 // Clean up after IE to avoid memory leaks
-if ($.browser.msie) $(window).unload(function() {
+if (jQuery.browser.msie) jQuery(window).unload(function() {
        var event = jQuery.event, global = event.global;
        for (var type in global) {
                var els = global[type], i = els.length;
-               if (i>0) do event.remove(els[i-1], type); while (--i);
+               if (i>0) do if (type != 'unload') event.remove(els[i-1], type); while (--i);
        }
 });