Changeset 4287

Timestamp:

12/20/07 20:40:20 (9 months ago)

Author:

davidserduke

Message:

Fixed a couple problems found in the test suite. First, IE doesn't like it when text nodes were trying to duplicate their events in clone, so don't do text nodes (that shouldn't have events anyway). Also the fx module was freezing from a recent update that wasn't quite finished.

Location:

trunk/jquery

Files:

• src/core.js (modified) (1 diff)
• test/unit/fx.js (modified) (2 diffs)

trunk/jquery/src/core.js

@@ -324 +324 @@
         if ( events === true )
             this.find("*").andSelf().each(function(i){
+                if (this.nodeType == 3)
+                    return;
                 var events = jQuery.data( this, "events" );
 

trunk/jquery/test/unit/fx.js

@@ -2 +2 @@
 
 test("animate(Hash, Object, Function)", function() {
-    expect(3);
+    expect(1);
     stop();
     var hash = {opacity: 'show'};
@@ -8 +8 @@
     $('#foo').animate(hash, 0, function() {
         ok( hash.opacity == hashCopy.opacity, 'Check if animate changed the hash parameter' );
+        start();
     });
 });