Você só precisa atravessar cada li (Oops! a) até achar algum deles que contém um href igual ao location.pathname
. Portanto, há uma forma melhor de fazer isso se você re-interpretar o href de cada li, transformando-o em uma nova instância de Location
.
E por que adicionar um '/' no começo de cada diretório?
Porque a comparação de cada diretório contra o diretório de location.href
poderia falhar se não eu adicionasse, por exemplo, se tal diretório fosse igual à location.href
, sua comparação ainda falharia se não tivesse um '/' no começo, assim como location.href
poderia ter.
(location.href.pathname = "/dir";
(this.pathname = "dir")
(location.href.pathname = "/dir") === (this.pathname = "dir")
// false
Para saber se o diretório não contém um '/' no começo, basta uma condição como essa: pathname.charAt(0) !== '/'
, desde que o diretório é uma String
... ;]
$(document).ready(function() {
var pathname = location.pathname.charAt(0) !== '/' ?
location.pathname = '/' + location.pathname :
location.pathname
$('.navbar-nav a').each(function() {
var link = document.createElement('a')
link.href = this.href
if ((link.pathname.charAt(0) !== '/' ?
href = '/' + href :
href) === pathname) {
$(this).addClass('active')
return false
}
});
});