Skip to content

Commit

Permalink
Revised the Nokia support fallback. It turns out that Nokia supports …
Browse files Browse the repository at this point in the history
…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.
  • Loading branch information
scottjehl committed Jan 19, 2011
1 parent cb1f7ee commit 2b64b1d
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/dimensions.js
Expand Up @@ -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 ) {
Expand Down

0 comments on commit 2b64b1d

Please sign in to comment.