library-mirrors/jquery
1
0
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2024-11-23 02:54:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fixes #7912. Make sure .cur() only returns 0 as fallback value when it needs to ("", auto, undefined, null).

Browse Source 
This change makes .cur() more .cssHooks friendly. .cur() now returns the unmodified value by
.css() if it isn't a number, number-alike or a value that needs a fallback to 0.
This way fx.start doesn't need to be recalculated for complex values.
This commit is contained in:
louisremi 2011-02-17 17:26:23 +01:00 committed by Anton M
parent b9f5e2b974
commit 85d9343271
2 changed files with 43 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/effects.js vendored
View File

@ -185,7 +185,7 @@ jQuery.fn.extend({
} else { } else {
var parts = rfxnum.exec(val), var parts = rfxnum.exec(val),
start = e.cur() || 0; start = e.cur();
if ( parts ) { if ( parts ) {
var end = parseFloat( parts[2] ), var end = parseFloat( parts[2] ),
@ -336,8 +336,12 @@ jQuery.fx.prototype = {
return this.elem[ this.prop ]; return this.elem[ this.prop ];
} }
var r = parseFloat( jQuery.css( this.elem, this.prop ) ); var parsed,
return r || 0; r = jQuery.css( this.elem, this.prop );
// Empty strings, null, undefined and "auto" are converted to 0,
// complex values such as "rotate(1rad)" are returned as is,
// simple values such as "10px" are parsed to Float.
return isNaN( parsed = parseFloat( r ) ) ? !r || r === "auto" ? 0 : r : parsed;
}, },
// Start an animation from one number to another // Start an animation from one number to another

49
test/unit/effects.js vendored
View File

@ -558,21 +558,44 @@ jQuery.checkOverflowDisplay = function(){
start(); start();
} }
test("support negative values < -10000 (bug #7193)", function () { test("jQuery.fx.prototype.cur()", function() {
expect(1); expect(5);
stop(); var nothiddendiv = jQuery('#nothiddendiv').css({
color: '#ABC',
border: '5px solid black',
left: 'auto',
marginBottom: '11000px'
})[0];
jQuery.extend(jQuery.fx.step, { equals(
"marginBottom": function(fx) { (new jQuery.fx( nothiddendiv, {}, 'color' )).cur(),
equals( fx.cur(), -11000, "Element has margin-bottom of -11000" ); jQuery.css( nothiddendiv,'color' ),
delete jQuery.fx.step.marginBottom; "Return the same value as jQuery.css for complex properties (bug #7912)"
} );
});
jQuery("#main").css("marginBottom", "-11000px").animate({ marginBottom: "-11001px" }, { strictEqual(
duration: 1, (new jQuery.fx( nothiddendiv, {}, 'borderLeftWidth' )).cur(),
complete: start 5,
}); "Return simple values parsed as Float"
);
strictEqual(
(new jQuery.fx( nothiddendiv, {}, 'backgroundPosition' )).cur(),
0,
'Return 0 when jQuery.css returns an empty string'
);
strictEqual(
(new jQuery.fx( nothiddendiv, {}, 'left' )).cur(),
0,
'Return 0 when jQuery.css returns "auto"'
);
equals(
(new jQuery.fx( nothiddendiv, {}, 'marginBottom' )).cur(),
11000,
"support negative values < -10000 (bug #7193)"
);
}); });
test("JS Overflow and Display", function() { test("JS Overflow and Display", function() {