From 504e5995692a9d4444f8dd78c610f2fa5355fffb Mon Sep 17 00:00:00 2001 From: gnarf Date: Sun, 27 Mar 2011 06:32:04 -0500 Subject: [PATCH] effects.tests: changing the way the effects test runs - using delay() instead of callbacks can help us see if we have places where we destroyed the queue --- tests/visual/effects.all.js | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/tests/visual/effects.all.js b/tests/visual/effects.all.js index 5e47a4f48..f678c6276 100644 --- a/tests/visual/effects.all.js +++ b/tests/visual/effects.all.js @@ -14,12 +14,14 @@ $(function() { $(el).bind("click", function() { - $(this).addClass("current").hide(n, o, duration, function() { - var self = this; - window.setTimeout(function() { - $(self).show(n, o, duration, function() { $(this).removeClass("current"); }); - }, wait); - }); + $(this).addClass("current") + // delaying the initial animation makes sure that the queue stays in tact + .delay( 10 ) + .hide( n, o, duration ) + .delay( wait ) + .show( n, o, duration, function() { + $( this ).removeClass("current"); + }); }); }; -- 2.39.5