Changeset 5609

Timestamp:

05/15/08 14:45:29 (5 months ago)

Author:

aflesler

Message:

jquery core: SHOULD fix #2802. Caching document.defaultView, instead of defaultView.getComputedStyle as it fails on Safari 2.

Files:

• trunk/jquery/src/core.js (modified) (4 diffs)

trunk/jquery/src/core.js

@@ -597 +597 @@
 
 var expando = "jQuery" + now(), uuid = 0, windowData = {},
-
-// exclude the following css properties to add px
+    // exclude the following css properties to add px
     exclude = /z-?index|font-?weight|opacity|zoom|line-?height/i,
-// cache getComputedStyle
-    getComputedStyle = document.defaultView && document.defaultView.getComputedStyle;
+    // cache defaultView
+    defaultView = document.defaultView || {};
 
 jQuery.extend({
@@ -831 +830 @@
                 return false;
 
-            // getComputedStyle is cached
-            var ret = getComputedStyle( elem, null );
+            // defaultView is cached
+            var ret = defaultView.getComputedStyle( elem, null );
             return !ret || ret.getPropertyValue("color") == "";
         }
@@ -858 +857 @@
             ret = style[ name ];
 
-        else if ( getComputedStyle ) {
+        else if ( defaultView.getComputedStyle ) {
 
             // Only "float" is needed here
@@ -866 +865 @@
             name = name.replace( /([A-Z])/g, "-$1" ).toLowerCase();
 
-            var computedStyle = getComputedStyle( elem, null );
+            var computedStyle = defaultView.getComputedStyle( elem, null );
 
             if ( computedStyle && !color( elem ) )