From: fat <jacobthornton@gmail.com>
Date: Wed, 11 Jun 2014 00:37:40 +0000 (-0700)
Subject: fix #13220 Scrollspy generates wrong offsets for a page-- recalculate offsets if... 
X-Git-Tag: v3.2.0~83^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F13783%2Fhead;p=thirdparty%2Fbootstrap.git

fix #13220 Scrollspy generates wrong offsets for a page-- recalculate offsets if `scrollheight` changes
---

diff --git a/js/scrollspy.js b/js/scrollspy.js
index 78858680e3..fe8fbc6826 100644
--- a/js/scrollspy.js
+++ b/js/scrollspy.js
@@ -27,6 +27,7 @@
     this.offsets        = []
     this.targets        = []
     this.activeTarget   = null
+    this.scrollHeight   = 0
 
     this.$scrollElement.on('scroll.bs.scrollspy', process)
     this.refresh()
@@ -39,6 +40,10 @@
     offset: 10
   }
 
+  ScrollSpy.prototype.getScrollHeight = function () {
+    return this.$scrollElement[0].scrollHeight || Math.max(this.$body[0].scrollHeight, document.documentElement.scrollHeight)
+  }
+
   ScrollSpy.prototype.refresh = function () {
     var offsetMethod = 'offset'
     var offsetBase   = 0
@@ -50,6 +55,7 @@
 
     this.offsets = []
     this.targets = []
+    this.scrollHeight = this.getScrollHeight()
 
     var self     = this
 
@@ -74,13 +80,17 @@
 
   ScrollSpy.prototype.process = function () {
     var scrollTop    = this.$scrollElement.scrollTop() + this.options.offset
-    var scrollHeight = this.$scrollElement[0].scrollHeight || Math.max(this.$body[0].scrollHeight, document.documentElement.scrollHeight)
+    var scrollHeight = this.getScrollHeight()
     var maxScroll    = this.options.offset + scrollHeight - this.$scrollElement.height()
     var offsets      = this.offsets
     var targets      = this.targets
     var activeTarget = this.activeTarget
     var i
 
+    if (this.scrollHeight != scrollHeight) {
+      this.refresh()
+    }
+
     if (scrollTop >= maxScroll) {
       return activeTarget != (i = targets[targets.length - 1]) && this.activate(i)
     }