Skip to content
Permalink
Browse files

Fixed a couple problems found in the test suite. First, IE doesn't li…

…ke 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.
  • Loading branch information
davids549 committed Dec 20, 2007
1 parent bdd6aca commit 50c40add5ab530cefbdd09dbdcb641f22b9dc26c
Showing with 4 additions and 1 deletion.
  1. +2 −0 src/core.js
  2. +2 −1 test/unit/fx.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 )
@@ -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();
});
});

0 comments on commit 50c40ad

Please sign in to comment.
You can’t perform that action at this time.