From: Richard Gibson Date: Tue, 14 May 2013 00:37:42 +0000 (-0400) Subject: Ref 3971c2eb: Sync with 1.x-master X-Git-Tag: 2.0.1~9 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d80c6aca18240281f553a35d727c2891d4bc2b57;p=jquery.git Ref 3971c2eb: Sync with 1.x-master --- diff --git a/src/effects.js b/src/effects.js index bb10b73f8..78f36827a 100644 --- a/src/effects.js +++ b/src/effects.js @@ -249,7 +249,7 @@ jQuery.Animation = jQuery.extend( Animation, { function defaultPrefilter( elem, props, opts ) { /* jshint validthis: true */ - var index, prop, value, toggle, tween, hooks, oldfire, + var prop, value, toggle, tween, hooks, oldfire, anim = this, orig = {}, style = elem.style, @@ -310,21 +310,21 @@ function defaultPrefilter( elem, props, opts ) { // show/hide pass - for ( index in props ) { - value = props[ index ]; + for ( prop in props ) { + value = props[ prop ]; if ( rfxtypes.exec( value ) ) { - delete props[ index ]; + delete props[ prop ]; toggle = toggle || value === "toggle"; if ( value === ( hidden ? "hide" : "show" ) ) { // If there is dataShow left over from a stopped hide or show and we are going to proceed with show, we should pretend to be hidden - if ( value === "show" && dataShow && dataShow[ index ] !== undefined ) { + if ( value === "show" && dataShow && dataShow[ prop ] !== undefined ) { hidden = true; } else { continue; } } - orig[ index ] = dataShow && dataShow[ index ] || jQuery.style( elem, index ); + orig[ prop ] = dataShow && dataShow[ prop ] || jQuery.style( elem, prop ); } } diff --git a/test/unit/effects.js b/test/unit/effects.js index 47bc9b4b6..d59ff84a6 100644 --- a/test/unit/effects.js +++ b/test/unit/effects.js @@ -1379,8 +1379,8 @@ test("line-height animates correctly (#13855)", function() { animated = jQuery( "

unitless

" + "

px

" + - "

percent

" + - "

em

" + "

percent

" + + "

em

" ).appendTo("#qunit-fixture"), initialHeight = jQuery.map( animated, function( el ) { return jQuery( el ).height();