]> source.dussan.org Git - jquery.git/commitdiff
Bug 7345; Add support for explicit/relative string values in .css - modified from...
authorDan Heberden <danheberden@gmail.com>
Mon, 4 Apr 2011 18:21:15 +0000 (11:21 -0700)
committerDan Heberden <danheberden@gmail.com>
Mon, 4 Apr 2011 18:21:15 +0000 (11:21 -0700)
src/css.js
test/unit/css.js

index 17ac136bbd53ef263ebe1ad4da8d63a58d4d3763..d23549fa4f08b5be3cf5b84d16780d11e407bed6 100644 (file)
@@ -7,6 +7,8 @@ var ralpha = /alpha\([^)]*\)/i,
        rupper = /([A-Z]|^ms)/g,
        rnumpx = /^-?\d+(?:px)?$/i,
        rnum = /^-?\d/,
+       rrelNum = /=/,
+       rrelString = /[^+\-\de]+/g,
 
        cssShow = { position: "absolute", visibility: "hidden", display: "block" },
        cssWidth = [ "Left", "Right" ],
@@ -75,7 +77,7 @@ jQuery.extend({
                }
 
                // Make sure that we're working with the right name
-               var ret, origName = jQuery.camelCase( name ),
+               var ret, parsed, type, origName = jQuery.camelCase( name ),
                        style = elem.style, hooks = jQuery.cssHooks[ origName ];
 
                name = jQuery.cssProps[ origName ] || origName;
@@ -87,6 +89,12 @@ jQuery.extend({
                                return;
                        }
 
+                       // convert string to number or relative number
+                       if ( type === "string" ) {
+                               parsed = +value.replace( rrelString, '' );
+                               value = rrelNum.test( value ) ? parsed + jQuery.css( elem, name ) : parsed;
+                       }
+
                        // If a number was passed in, add 'px' to the (except for certain CSS properties)
                        if ( typeof value === "number" && !jQuery.cssNumber[ origName ] ) {
                                value += "px";
index 8ae8fcb346848b8c028951dd3ecc99c373e2494f..c4724fcd05053860fbc54ba2de090d81d0a302fb 100644 (file)
@@ -105,6 +105,27 @@ test("css(String|Hash)", function() {
        equals( child[0].style.fontSize, old, "Make sure font-size isn't changed on null." );
 });
 
+test("css() explicit and relative values", function() {
+       var $elem = jQuery('#nothiddendiv');
+       
+       $elem.css({ width: 1, height: 1 });
+       ok( [$elem.width(), $elem.height()], [1,1] );
+       $elem.css({ width: "+=9", height: "+=9" });
+       ok( [$elem.width(), $elem.height()], [10,10] );
+       $elem.css({ width: "-=9", height: "-=9" });
+       ok( [$elem.width(), $elem.height()], [1,1] );
+       $elem.css({ width: "+=9px", height: "+=9px" });
+       ok( [$elem.width(), $elem.height()], [10,10] );
+       $elem.css({ width: "-=9px", height: "-=9px" });
+       ok( [$elem.width(), $elem.height()], [1,1] );
+       $elem.css("width", "+=9").css("height", "+=9");
+       ok( [$elem.width(), $elem.height()], [10,10] );
+       $elem.css("width", "+9").css("height", "+9");
+       ok( [$elem.width(), $elem.height()], [9,9] );
+       $elem.css("width", "-9").css("height", "-9");
+       ok( [$elem.width(), $elem.height()], [0,0] );
+});
+
 test("css(String, Object)", function() {
        expect(22);