fixed #8715

edaeb338 · Jacob Thornton · a31c5cea · edaeb338
Commit edaeb338 authored 11 years ago by Jacob Thornton
Hide whitespace changes
Inline Side-by-side

Showing

with 5 additions and 6 deletions
+5 -6
--- a/assets/js/application.js
+++ b/assets/js/application.js
@@ -7,10 +7,11 @@
  $(function(){

    var $window = $(window)
+    var $body   = $(document.body)

    var navHeight = $('.navbar').outerHeight(true) + 10

-    $(document.body).scrollspy({
+    $body.scrollspy({
      target: '.bs-sidebar',
      offset: navHeight
    })
@@ -19,14 +20,12 @@
      e.preventDefault()
    })

-    $(document.body).on('click', '.bs-sidenav [href^=#]', function (e) {
+    $body.on('click', '.bs-sidenav [href^=#]', function (e) {
      var $target = $(this.getAttribute('href'))

-      // e.preventDefault() // prevent browser scroll
+      e.preventDefault() // prevent browser scroll

-      document.body.scrollTop =
-        $target.offset().top -
-        navHeight + 5 // offset scroll by nav
+      $window.scrollTop($target.offset().top - navHeight + 5)
    })

    // back to top