You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by hs...@apache.org on 2011/09/06 22:53:40 UTC

svn commit: r1165834 - /shindig/trunk/features/src/main/javascript/features/dynamic-width.width/dynamic-width-width.js

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;