Made a number of spacing changes to bring the code more-inline with the jQuery Core...

[jquery.git] / src / css.js

diff --git a/src/css.js b/src/css.js
index ddd42b4..e52e453 100644 (file)
--- a/src/css.js
+++ b/src/css.js
@@ -5,21 +5,25 @@ var rexclude = /z-?index|font-?weight|opacity|zoom|line-?height/i,
        rfloat = /float/i,
        rdashAlpha = /-([a-z])/ig,
        rupper = /([A-Z])/g,
-       rnumpx = /^\d+(?:px)?$/i,
-       rnum = /^\d/,
+       rnumpx = /^-?\d+(?:px)?$/i,
+       rnum = /^-?\d/,
+
+       cssShow = { position: "absolute", visibility: "hidden", display:"block" },
+       cssWidth = [ "Left", "Right" ],
+       cssHeight = [ "Top", "Bottom" ],
 
        // cache check for defaultView.getComputedStyle
        getComputedStyle = document.defaultView && document.defaultView.getComputedStyle,
        // normalize float css property
        styleFloat = jQuery.support.cssFloat ? "cssFloat" : "styleFloat",
-       fcamelCase = function(all, letter){
+       fcamelCase = function( all, letter ) {
                return letter.toUpperCase();
        };
 
 jQuery.fn.css = function( name, value ) {
        return access( this, name, value, true, function( elem, name, value ) {
-               if (value === undefined) {
-                       return jQuery.css( elem, name );
+               if ( value === undefined ) {
+                       return jQuery.curCSS( elem, name );
                }
                
                if ( typeof value === "number" && !rexclude.test(name) ) {
@@ -52,8 +56,9 @@ jQuery.extend({
                                style.zoom = 1;
 
                                // Set the alpha filter to set the opacity
-                               style.filter = (style.filter || "").replace( ralpha, "" ) +
-                                       (parseInt( value ) + '' === "NaN" ? "" : "alpha(opacity=" + value * 100 + ")");
+                               var opacity = parseInt( value, 10 ) + '' === "NaN" ? "" : "alpha(opacity=" + value * 100 + ")";
+                               var filter = style.filter || jQuery.curCSS( elem, 'filter' ) || "";
+                               style.filter = ralpha.test(filter) ? filter.replace(ralpha, opacity) : opacity;
                        }
 
                        return style.filter && style.filter.indexOf("opacity=") >= 0 ?
@@ -77,12 +82,14 @@ jQuery.extend({
 
        css: function( elem, name, force, extra ) {
                if ( name === "width" || name === "height" ) {
-                       var val, props = { position: "absolute", visibility: "hidden", display:"block" }, which = name === "width" ? [ "Left", "Right" ] : [ "Top", "Bottom" ];
+                       var val, props = cssShow, which = name === "width" ? cssWidth : cssHeight;
 
                        function getWH() {
                                val = name === "width" ? elem.offsetWidth : elem.offsetHeight;
 
-                               if ( extra === "border" ) { return; }
+                               if ( extra === "border" ) {
+                                       return;
+                               }
 
                                jQuery.each( which, function() {
                                        if ( !extra ) {
@@ -140,7 +147,13 @@ jQuery.extend({
 
                        name = name.replace( rupper, "-$1" ).toLowerCase();
 
-                       var computedStyle = elem.ownerDocument.defaultView.getComputedStyle( elem, null );
+                       var defaultView = elem.ownerDocument.defaultView;
+
+                       if ( !defaultView ) {
+                               return null;
+                       }
+
+                       var computedStyle = defaultView.getComputedStyle( elem, null );
 
                        if ( computedStyle ) {
                                ret = computedStyle.getPropertyValue( name );
@@ -199,17 +212,18 @@ jQuery.extend({
 });
 
 if ( jQuery.expr && jQuery.expr.filters ) {
-       jQuery.expr.filters.hidden = function(elem){
+       jQuery.expr.filters.hidden = function( elem ) {
                var width = elem.offsetWidth, height = elem.offsetHeight,
-                        force = /^tr$/i.test( elem.nodeName ); // ticket #4512
-               return ( width === 0 && height === 0 && !force ) ?
+                       skip = elem.nodeName.toLowerCase() === "tr";
+
+               return width === 0 && height === 0 && !skip ?
                        true :
-                               ( width !== 0 && height !== 0 && !force ) ?
-                                       false :
-                                               !!( jQuery.curCSS(elem, "display") === "none" );
+                       width > 0 && height > 0 && !skip ?
+                               false :
+                               jQuery.curCSS(elem, "display") === "none";
        };
 
-       jQuery.expr.filters.visible = function(elem){
-               return !jQuery.expr.filters.hidden(elem);
+       jQuery.expr.filters.visible = function( elem ) {
+               return !jQuery.expr.filters.hidden( elem );
        };
 }