Merge pull request #15378 from twbs/fix-ie-modal-scrollbar-for-realz

Modal: Work around IE scrollbars not taking up page width

1 files changed, 6 insertions, 1 deletions

diff --git a/js/modal.js b/js/modal.js
index e7704b8fc..c45437dfb 100644
--- a/js/modal.js
+++ b/js/modal.js
@@ -248,7 +248,12 @@
   }
 
   Modal.prototype.checkScrollbar = function () {
-    this.bodyIsOverflowing = document.body.scrollHeight > document.documentElement.clientHeight
+    var fullWindowWidth = window.innerWidth
+    if (!fullWindowWidth) { // workaround for missing window.innerWidth in IE8
+      var documentElementRect = document.documentElement.getBoundingClientRect()
+      fullWindowWidth = documentElementRect.right - Math.abs(documentElementRect.left)
+    }
+    this.bodyIsOverflowing = document.body.clientWidth < fullWindowWidth
     this.scrollbarWidth = this.measureScrollbar()
   }