Scrollspy selecting the last element when at the bottom of the page (f…

…ixes #17739)
twbs · Nov 26, 2016 · 39d7861 · 39d7861
1 parent 8a7ce25
commit 39d7861
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/js/src/scrollspy.js b/js/src/scrollspy.js
@@ -196,12 +196,17 @@ const ScrollSpy = (($) => {
       )
     }
 
+    _getOffsetHeight() {
+      return this._scrollElement === window ?
+          window.innerHeight : this._scrollElement.offsetHeight
+    }
+
     _process() {
       const scrollTop    = this._getScrollTop() + this._config.offset
       const scrollHeight = this._getScrollHeight()
       const maxScroll    = this._config.offset
         + scrollHeight
-        - this._scrollElement.offsetHeight
+        - this._getOffsetHeight()
 
       if (this._scrollHeight !== scrollHeight) {
         this.refresh()
@@ -213,6 +218,7 @@ const ScrollSpy = (($) => {
         if (this._activeTarget !== target) {
           this._activate(target)
         }
+        return
       }
 
       if (this._activeTarget && scrollTop < this._offsets[0]) {