Author: hsaputra Date: Tue Sep 6 20:53:39 2011 New Revision: 1165834 URL: http://svn.apache.org/viewvc?rev=1165834&view=rev Log: SHINDIG-1614 | vw is undefined in dynamic-width feature | Patch from Dan Dumont. Thanks!
Code review: https://reviews.apache.org/r/1729/ Modified: shindig/trunk/features/src/main/javascript/features/dynamic-width.width/dynamic-width-width.js Modified: shindig/trunk/features/src/main/javascript/features/dynamic-width.width/dynamic-width-width.js URL: http://svn.apache.org/viewvc/shindig/trunk/features/src/main/javascript/features/dynamic-width.width/dynamic-width-width.js?rev=1165834&r1=1165833&r2=1165834&view=diff ============================================================================== --- shindig/trunk/features/src/main/javascript/features/dynamic-width.width/dynamic-width-width.js (original) +++ shindig/trunk/features/src/main/javascript/features/dynamic-width.width/dynamic-width-width.js Tue Sep 6 20:53:39 2011 @@ -50,8 +50,7 @@ gadgets.window = gadgets.window || {}; // document.documentElement.offsetWidth // Based on studying the values output by different browsers, // use the value that's NOT equal to the viewport width found above. - return docEl.scrollWidth !== vw ? docEl.scrollWidth : docEl.offsetWidth; - + return docEl.scrollWidth !== viewportWidth ? docEl.scrollWidth : docEl.offsetWidth; } else if (body) { var scrollWidth = body.scrollWidth; var offsetWidth = body.offsetWidth;
