fix for #4234. hover can take one function to use for both enter and leave.

diff --git a/src/event.js b/src/event.js
index 8d737bf4bb3e84b7e1c81c00dc4a8089da0928ec..ebf1231cba0b43f95d04a7beb4f6bf947a6ea774 100644 (file)
--- a/src/event.js
+++ b/src/event.js
@@ -569,7 +569,7 @@ jQuery.fn.extend({
        },
 
        hover: function( fnOver, fnOut ) {
-               return this.mouseenter( fnOver ).mouseleave( fnOut );
+               return this.mouseenter( fnOver ).mouseleave( fnOut || fnOver );
        },
 
        ready: function( fn ) {

diff --git a/test/unit/event.js b/test/unit/event.js
index 63c640d1195a61a17c0f7647b20171c8c52c74f3..98833ece87d9267fb025ae840a92be19467b7972 100644 (file)
--- a/test/unit/event.js
+++ b/test/unit/event.js
@@ -233,6 +233,24 @@ test("unbind(eventObject)", function() {
        assert( 0 );
 });
 
+test("hover()", function() {
+       var times = 0,
+               handler1 = function( event ) { ++times; },
+               handler2 = function( event ) { ++times; };
+
+       jQuery("#firstp")
+               .hover(handler1, handler2)
+               .mouseenter().mouseleave()
+               .unbind("mouseenter", handler1)
+               .unbind("mouseleave", handler2)
+               .hover(handler1)
+               .mouseenter().mouseleave()
+               .unbind("mouseenter mouseleave", handler1)
+               .mouseenter().mouseleave();
+
+       equals( times, 4, "hover handlers fired" );
+});
+
 test("trigger() shortcuts", function() {
        expect(6);
        jQuery('<li><a href="#">Change location</a></li>').prependTo('#firstUL').find('a').bind('click', function() {