From 6fd5e480c1cd3e481e7097763ee281b65bc74306 Mon Sep 17 00:00:00 2001 From: Richard Gibson Date: Sat, 25 May 2013 10:18:57 -0400 Subject: Fix #13937: Correctly scope .finish() following multi-element .animate(). Thanks @gnarf37. Close gh-1279. (cherry picked from commit ae9e05e9f3cb071232b056005755acb5926e403e) --- src/effects.js | 8 +++----- src/queue.js | 1 - 2 files changed, 3 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/effects.js b/src/effects.js index cb78b5cfe..b0a9083e7 100644 --- a/src/effects.js +++ b/src/effects.js @@ -488,9 +488,7 @@ jQuery.fn.extend({ doAnimation = function() { // Operate on a copy of prop so per-property easing won't be lost var anim = Animation( this, jQuery.extend( {}, prop ), optall ); - doAnimation.finish = function() { - anim.stop( true ); - }; + // Empty animations, or finishing resolves immediately if ( empty || data_priv.get( this, "finish" ) ) { anim.stop( true ); @@ -570,8 +568,8 @@ jQuery.fn.extend({ // empty the queue first jQuery.queue( this, type, [] ); - if ( hooks && hooks.cur && hooks.cur.finish ) { - hooks.cur.finish.call( this ); + if ( hooks && hooks.stop ) { + hooks.stop.call( this, true ); } // look for any active animations, and finish them diff --git a/src/queue.js b/src/queue.js index 46b9ba8fc..1297bc1b4 100644 --- a/src/queue.js +++ b/src/queue.js @@ -35,7 +35,6 @@ jQuery.extend({ startLength--; } - hooks.cur = fn; if ( fn ) { // Add a progress sentinel to prevent the fx queue from being -- cgit v1.2.3