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;