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;


Reply via email to