Merge pull request #28 from seutje/patch-1

Tolerance causes the highlighting of nav elements to seem inaccurate
h5bp · Dec 2, 2011 · f7c0cfd · f7c0cfd
2 parents dd94613 + e826324
commit f7c0cfd
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/js/init.js b/js/init.js
@@ -143,13 +143,11 @@
 			timeout = false, // so gonna clear this later, you have NO idea
 			last = false, // makes sure the previous link gets un-activated
 			check = function() {
-				var scroll = $scrollable.scrollTop(),
-					height = $body.height(),
-					tolerance = $window.height() * ( scroll / height );
+				var scroll = $scrollable.scrollTop();
 
 				$.each( cache, function( i, v ) {
 					// if we're past the link's section, activate it
-					if ( scroll + tolerance >  v.target.position().top  ) {
+					if ( scroll + $navheight >  v.target.position().top  ) {
 						last && last.removeClass('active');
 						last = v.link.addClass('active');
 					} else {