Merge pull request #66 from theonion/bug/use-window-innerwidth

theonion · Jun 19, 2018 · f091eb1 · f091eb1
2 parents 64ec5b5 + 0456a28
commit f091eb1
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/manager.js b/src/manager.js
@@ -29,7 +29,7 @@ var AdManager = function (options) {
   this.adId = 0;
   this.initialized = false;
   this.viewportWidth = 0;
-  this.oldViewportWidth = window.document.body.clientWidth;
+  this.oldViewportWidth = window.innerWidth;
   this.options = utils.extend(defaultOptions, options);
   this.bindContext();
 
@@ -90,7 +90,7 @@ AdManager.prototype.handleWindowResize = function () {
     return;
   }
 
-  this.viewportWidth = window.document.body.clientWidth;
+  this.viewportWidth = window.innerWidth;
 
   if (!this.oldViewportWidth || this.oldViewportWidth !== this.viewportWidth) {
     // viewport size has actually changed, reload ads
@@ -314,7 +314,7 @@ AdManager.prototype.generateId = function () {
   * @returns {Integer} client width in pixels
 */
 AdManager.prototype.getClientWidth = function () {
-  return window.document.body.clientWidth;
+  return window.innerWidth;
 };
 
 /**