Fixed #11 - The scrollTop is now correct for when there is a hash on the url

gfranko · gfranko · commit db0a62234c82 · 2012-12-23T16:16:43.000-05:00
diff --git a/demos/default.html b/demos/default.html
@@ -47,17 +47,7 @@
   </head>
 
   <body>
-<nav class="navbar">
-  <div class="navbar-inner">
-    <div class="container">
-      <ul class="nav">
-      <li><a href="/alatriste/1">One</a></li>
-      <li class="active"><a href="/alatriste/2">Two</li>
-      <li><a href="/alatriste/3">Three</li>
-      </ul>
-    </div>
-  </div>
-</nav>
+
 <a href="https://github.com/gfranko/jquery.tocify.js" target="_blank" id="fork-me"><img style="position: fixed;top: 0; right: 0; border: 0;z-index:999999;" src="https://s3.amazonaws.com/github/ribbons/forkme_right_white_ffffff.png" alt="Fork me on GitHub" alt="Fork me on GitHub"></a>
 
     <div class="container-fluid">
diff --git a/src/javascripts/jquery.tocify.js b/src/javascripts/jquery.tocify.js
@@ -117,13 +117,13 @@
             // Adds jQuery event handlers to the newly generated table of contents
             self._setEventHandlers();
 
-            // The setTimeout allows the correct offset() to be calculated
-            setTimeout(function() {
+            // The window onload events allows enough time for the correct offset to be calculated
+            window.addEventListener("load", function() {
 
-                // Set the active TOC item
+                // Sets the active TOC item
                 self._setActiveElement();
 
-            }, 0);
+            }, false);
 
         },
 
diff --git a/src/javascripts/jquery.tocify.min.js b/src/javascripts/jquery.tocify.min.js
