Fix dangling comma of death in data.js

[jquery.git] / src / css.js

diff --git a/src/css.js b/src/css.js
index b8a1350..94a021b 100644 (file)
--- a/src/css.js
+++ b/src/css.js
@@ -8,10 +8,6 @@ var exclude = /z-?index|font-?weight|opacity|zoom|line-?height/i,
 jQuery.fn.css = function( name, value ) {
        var options = name, isFunction = jQuery.isFunction( value );
 
-       // ignore negative width and height values
-       if ( (name == 'width' || name == 'height') && parseFloat(value) < 0 )
-               value = undefined;
-
        if ( typeof name === "string" ) {
                // Are we setting the style?
                if ( value === undefined ) {
@@ -25,6 +21,13 @@ jQuery.fn.css = function( name, value ) {
                        options[ name ] = value;
                }
        }
+       
+       var isFunction = {};
+       
+       // For each value, determine whether it's a Function so we don't
+       // need to determine it again for each element
+       for ( var prop in options )
+               isFunction[prop] = jQuery.isFunction( options[prop] );
 
        // For each element...
        for ( var i = 0, l = this.length; i < l; i++ ) {
@@ -34,9 +37,7 @@ jQuery.fn.css = function( name, value ) {
                for ( var prop in options ) {
                        value = options[prop];
 
-                       if ( isFunction ) {
-                               value = value.call( elem, i );
-                       }
+                       if ( isFunction[prop] ) value = value.call( elem, i );
 
                        if ( typeof value === "number" && !exclude.test(prop) ) {
                                value = value + "px";
@@ -55,6 +56,10 @@ jQuery.extend({
                if (!elem || elem.nodeType == 3 || elem.nodeType == 8)
                        return undefined;
 
+               // ignore negative width and height values #1599
+               if ( (name == 'width' || name == 'height') && parseFloat(value) < 0 )
+                       value = undefined;
+
                var style = elem.style || elem, set = value !== undefined;
 
                // IE uses filters for opacity
@@ -120,12 +125,12 @@ jQuery.extend({
        },
 
        curCSS: function( elem, name, force ) {
-               var ret, style = elem.style;
+               var ret, style = elem.style, filter;
 
                // IE uses filters for opacity
-               if ( !jQuery.support.opacity && name == "opacity" ) {
-                       ret = style.filter && style.filter.indexOf("opacity=") >= 0 ?
-                               (parseFloat( style.filter.match(/opacity=([^)]*)/)[1] ) / 100) + '':
+               if ( !jQuery.support.opacity && name === "opacity" && elem.currentStyle ) {
+                       ret = (elem.currentStyle.filter || "").match(/opacity=([^)]*)/) ?
+                               (parseFloat(RegExp.$1) / 100) + "" :
                                "";
 
                        return ret === "" ?