Skip to content
Permalink
Browse files

Fixes #10021. Allow negative relative values for `.css()` (e.g., `"+=…

…-20px"`) since `.animate()` already allows it. Useful for when the relative value is a variable.
  • Loading branch information...
dmethvin committed Aug 16, 2011
1 parent b22c904 commit 015328787cd2e52efbc77d1254ebcac13cc4ac53
Showing with 15 additions and 10 deletions.
  1. +8 −9 src/css.js
  2. +7 −1 test/unit/css.js
@@ -6,8 +6,7 @@ var ralpha = /alpha\([^)]*\)/i,
rupper = /([A-Z]|^ms)/g,
rnumpx = /^-?\d+(?:px)?$/i,
rnum = /^-?\d/,
rrelNum = /^[+\-]=/,
rrelNumFilter = /[^+\-\.\de]+/g,
rrelNum = /^([+\-])=([+\-\.\de]+)/,

cssShow = { position: "absolute", visibility: "hidden", display: "block" },
cssWidth = [ "Left", "Right" ],
@@ -84,18 +83,18 @@ jQuery.extend({
if ( value !== undefined ) {
type = typeof value;

// Make sure that NaN and null values aren't set. See: #7116
if ( type === "number" && isNaN( value ) || value == null ) {
return;
}

// convert relative number strings (+= or -=) to relative numbers. #7345
if ( type === "string" && rrelNum.test( value ) ) {
value = +value.replace( rrelNumFilter, "" ) + parseFloat( jQuery.css( elem, name ) );
if ( type === "string" && (ret = rrelNum.exec( value )) ) {
value = (ret[1] === "+"? +ret[2] : -ret[2]) + parseFloat( jQuery.css( elem, name ) );
// Fixes bug #9237
type = "number";
}

// Make sure that NaN and null values aren't set. See: #7116
if ( type === "number" && isNaN( value ) || value == null ) {
return;
}

// If a number was passed in, add 'px' to the (except for certain CSS properties)
if ( type === "number" && !jQuery.cssNumber[ origName ] ) {
value += "px";
@@ -109,7 +109,7 @@ test("css(String|Hash)", function() {
});

test("css() explicit and relative values", function() {
expect(27);
expect(29);
var $elem = jQuery("#nothiddendiv");

$elem.css({ width: 1, height: 1, paddingLeft: "1px", opacity: 1 });
@@ -141,6 +141,12 @@ test("css() explicit and relative values", function() {
$elem.css( "width", "-=9px" );
equals( $elem.width(), 1, "'-=9px' on width (params)" );

$elem.css( "width", "-=-9px" );
equals( $elem.width(), 10, "'-=-9px' on width (params)" );

$elem.css( "width", "+=-9px" );
equals( $elem.width(), 1, "'+=-9px' on width (params)" );

$elem.css({ paddingLeft: "+=4" });
equals( $elem.css("paddingLeft"), "5px", "'+=4' on paddingLeft (hash)" );

0 comments on commit 0153287

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