IsmaelAli

It works =D Thank you thank you so much brother (y) =)

Don't for get your ul tag. The nav tag doesn't replace it

I think this what you are after
 
css change to add a .current rule
 
.IsmaelNavbar nav ul li a:hover,.IsmaelNavbar nav ul li a.current{ color:#fff; height:20px; border-bottom:4px solid #fff; padding-bottom:12px; } SmoothScroll edited to add class on click
 
$(document).ready(function() { function filterPath(string) { return string .replace(/^\//,'') .replace(/(index|default).[a-zA-Z]{3,4}$/,'') .replace(/\/$/,''); } var locationPath = filterPath(location.pathname); var scrollElem = scrollableElement('html', 'body'); $('a[href*=#]').each(function() { var thisPath = filterPath(this.pathname) || locationPath; if ( locationPath == thisPath && (location.hostname == this.hostname || !this.hostname) && this.hash.replace(/#/,'') ) { var $target = $(this.hash), target = this.hash; if (target) { var targetOffset = $target.offset().top; $(this).click(function(event) { event.preventDefault(); //removes current class $('a.current').removeClass('current'); //adds current class to clicked $(this).addClass('current'); $(scrollElem).animate({scrollTop: targetOffset}, 400, function() { location.hash = target; }); }); } } }); // use the first element that is "scrollable" function scrollableElement(els) { for (var i = 0, argLength = arguments.length; i <argLength; i++) { var el = arguments[i], $scrollElement = $(el); if ($scrollElement.scrollTop()> 0) { return el; } else { $scrollElement.scrollTop(1); var isScrollable = $scrollElement.scrollTop()> 0; $scrollElement.scrollTop(0); if (isScrollable) { return el; } } } return []; } }); Hth

When the links are clicked are you scrolling the current page or linking to a new page ?

You have class active in your CSS, but not in your HTML.
Add the class to the link you want active, for example:
 
<li><a class="active" href="#Home">Home</a></li> Are these tabs or are they going to be link to pages?
 
*Welcome to the forum