Changeset 4191

Timestamp:

12/16/07 22:18:35 (8 months ago)

Author:

brandon.aaron

Message:

Cleaned up mouseenter and mouseleave special events

Files:

• trunk/jquery/src/event.js (modified) (2 diffs)

trunk/jquery/src/event.js

@@ -367 +367 @@
         mouseenter: {
             setup: function() {
-                if (jQuery.browser.msie) return false;
-                jQuery(this).bind('mouseover', jQuery.event.special.mouseenter.handler);
+                if ( jQuery.browser.msie ) return false;
+                jQuery(this).bind("mouseover", jQuery.event.special.mouseenter.handler);
                 return true;
             },
         
             teardown: function() {
-                if (jQuery.browser.msie) return false;
-                jQuery(this).unbind('mouseover', jQuery.event.special.mouseenter.handler);
+                if ( jQuery.browser.msie ) return false;
+                jQuery(this).unbind("mouseover", jQuery.event.special.mouseenter.handler);
                 return true;
             },
             
             handler: function(event) {
-                var args = Array.prototype.slice.call( arguments, 1 );
                 // If we actually just moused on to a sub-element, ignore it
                 if ( withinElement(event, this) ) return true;
                 // Execute the right handlers by setting the event type to mouseenter
-                event.type = 'mouseenter';
-                // Include the event object as the first argument
-                args.unshift(event);
-                var val = jQuery.event.handle.apply(this, args);
-                return val;
+                arguments[0].type = "mouseenter";
+                return jQuery.event.handle.apply(this, arguments);
             }
         },
@@ -393 +389 @@
         mouseleave: {
             setup: function() {
-                if (jQuery.browser.msie) return false;
-                jQuery(this).bind('mouseout', jQuery.event.special.mouseleave.handler);
+                if ( jQuery.browser.msie ) return false;
+                jQuery(this).bind("mouseout", jQuery.event.special.mouseleave.handler);
                 return true;
             },
         
             teardown: function() {
-                if (jQuery.browser.msie) return false;
-                jQuery(this).unbind('mouseout', jQuery.event.special.mouseleave.handler);
+                if ( jQuery.browser.msie ) return false;
+                jQuery(this).unbind("mouseout", jQuery.event.special.mouseleave.handler);
                 return true;
             },
             
             handler: function(event) {
-                var args = Array.prototype.slice.call( arguments, 1 );
                 // If we actually just moused on to a sub-element, ignore it
                 if ( withinElement(event, this) ) return true;
                 // Execute the right handlers by setting the event type to mouseleave
-                event.type = 'mouseleave';
-                // Include the event object as the first argument
-                args.unshift(event);
-                var val = jQuery.event.handle.apply(this, args);
-                return val;
+                arguments[0].type = "mouseleave";
+                return jQuery.event.handle.apply(this, arguments);
             }
         }