From: scottjehl <scott@scottjehl.com>
Date: Wed, 19 Jan 2011 17:40:32 +0000 (-0500)
Subject: Revised the Nokia support fallback. It turns out that Nokia supports the documentElem... 
X-Git-Tag: 1.5rc1~19^2
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2b64b1db6877f52ea2958c6aafb0e4644fbcfe13;p=jquery.git

Revised the Nokia support fallback. It turns out that Nokia supports the documentElement property but does not define document.compatMode. Adding this third fallback allows Nokia to run jQuery error-free and return proper values for window width and height.
---

diff --git a/src/dimensions.js b/src/dimensions.js
index 9d8c35454..e2d411dd2 100644
--- a/src/dimensions.js
+++ b/src/dimensions.js
@@ -35,8 +35,10 @@ jQuery.each([ "Height", "Width" ], function( i, name ) {
 
 		if ( jQuery.isWindow( elem ) ) {
 			// Everyone else use document.documentElement or document.body depending on Quirks vs Standards mode
-			return elem.document.compatMode === "CSS1Compat" && elem.document.documentElement[ "client" + name ] ||
-				elem.document.body[ "client" + name ] || window.screen && window.screen[ name.toLowerCase() ];
+			// 3rd condition allows Nokia support, as it supports the docElem prop but not CSS1Compat
+			var docElemProp = elem.document.documentElement[ "client" + name ];
+			return elem.document.compatMode === "CSS1Compat" && docElemProp ||
+				elem.document.body[ "client" + name ] || docElemProp;
 
 		// Get document width or height
 		} else if ( elem.nodeType === 9 ) {