fix(mouseenter): FF no longer throws exceptions

angular · Apr 21, 2012 · 43d15f8 · 43d15f8
1 parent 1d26acb
commit 43d15f8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/jqLite.js b/src/jqLite.js
@@ -559,14 +559,12 @@ forEach({
           bindFn(element, 'mouseover', function(event) {
             counter++;
             if (counter == 1) {
-              event.type = 'mouseenter';
               mouseenter(event);
             }
           });
           bindFn(element, 'mouseout', function(event) {
             counter --;
             if (counter == 0) {
-              event.type = 'mouseleave';
               mouseleave(event);
             }
           });

diff --git a/test/jqLiteSpec.js b/test/jqLiteSpec.js
@@ -691,13 +691,13 @@ describe('jqLite', function() {
 
         parent.bind('mouseenter', function() { log += 'parentEnter;'; });
         parent.bind('mouseleave', function() { log += 'parentLeave;'; });
-        parent.mouseover = function(event) { parent.data('bind').mouseover(event || {}); };
-        parent.mouseout = function(event) { parent.data('bind').mouseout(event || {}); };
+        parent.mouseover = function() { browserTrigger(parent, 'mouseover'); };
+        parent.mouseout = function() { browserTrigger(parent, 'mouseout'); };
 
         child.bind('mouseenter', function() { log += 'childEnter;'; });
         child.bind('mouseleave', function() { log += 'childLeave;'; });
-        child.mouseover = function(event) { child.data('bind').mouseover(event || {}); };
-        child.mouseout = function(event) { child.data('bind').mouseout(event || {}); };
+        child.mouseover = function() { browserTrigger(child, 'mouseover'); };
+        child.mouseout = function() { browserTrigger(child, 'mouseout'); };
       });
 
       afterEach(function() {