From: David Serduke Date: Thu, 20 Dec 2007 20:40:20 +0000 (+0000) Subject: Fixed a couple problems found in the test suite. First, IE doesn't like it when... X-Git-Tag: 1.2.2~23 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=50c40add5ab530cefbdd09dbdcb641f22b9dc26c;p=jquery.git 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. --- diff --git a/src/core.js b/src/core.js index c2f94d2b7..10db3edaf 100644 --- a/src/core.js +++ b/src/core.js @@ -323,6 +323,8 @@ jQuery.fn = jQuery.prototype = { // Copy the events from the original to the clone if ( events === true ) this.find("*").andSelf().each(function(i){ + if (this.nodeType == 3) + return; var events = jQuery.data( this, "events" ); for ( var type in events ) diff --git a/test/unit/fx.js b/test/unit/fx.js index 5f6ad3993..1204e261c 100644 --- a/test/unit/fx.js +++ b/test/unit/fx.js @@ -1,12 +1,13 @@ module("fx"); test("animate(Hash, Object, Function)", function() { - expect(3); + expect(1); stop(); var hash = {opacity: 'show'}; var hashCopy = $.extend({}, hash); $('#foo').animate(hash, 0, function() { ok( hash.opacity == hashCopy.opacity, 'Check if animate changed the hash parameter' ); + start(); }); });