jQuery addClass() non sembra funzionare

Question

Desidero essere in grado di determinare su quale pagina l'utente è attualmente attivo. un semplice indicatore sotto forma di aggiungere uno stile unico al link di navigazione l'utente sta visualizzando il suo contesto

ecco il mio script, tutti i selettori funzionano bene, se anche l'istruzione fa il lavoro, ma la funzione addClass() non fa nulla .. qui è il mio codice:

<script> 
$(document).ready(function(){ 
    var pathname = window.location.pathname; 
    $("#links ul li a").each(function() { 
     var href= $(this).attr("href"); 
     if (pathname.indexOf(href) >= 0){ 
      $(this).addClass("active"); 
     } 
    }); 
}); 
</script> 


    <nav id="links"> 
    <ul> 
     <li><a href="index.php">Home</a></li> 
     <li><a>About</a></li> 
     <li><a>Contact</a></li> 
    </ul> 
</nav> 

HTML:

preso a lavorare, questo è il mio codice finalmente, v'è la necessità di eventuali migliorie?

$(document).ready(function(){ 
    var pathname = window.location.pathname; 
    var onDomain=true; 
    $("#links ul li a").each(function() { 
     var href= $(this).attr("href"); 
     if (href != undefined && pathname.indexOf(href) >= 0){ 
      $(this).addClass("active"); 
      onDomain=false; 
     } 
    }); 
    if (onDomain){ 
     $("#links ul li:first-child a").addClass("active"); 
    } 
}); 

Answer 1

Assegna href-a;

<nav id="links"> 
    <ul> 
     <li><a href="index.php">Home</a></li> 
     <li><a href="">About</a></li> 
     <li><a href="">Contact</a></li> 
    </ul> 
</nav> 

Quindi provare:

var pathname = window.location.pathname; 
    $("#links ul li a").each(function() { 
     var href= $(this).attr("href"); 
     if (href.length && pathname.indexOf(href) >= 0){ 
      $(this).addClass("active"); 
     } 
    }); 

Se non è possibile modificare il codice HTML quindi provare come segue:

var pathname = window.location.pathname; 
    $("#links ul li a").each(function() { 
     var href= $(this).attr("href"); 
     if (href != undefined && pathname.indexOf(href) >= 0){ 
      $(this).addClass("active"); 
     } 
    }); 

Answer 2

var loc = ''+($(location).attr('href').split('/').slice(-1)); 
if(typeof loc !== 'undefined'){ 
    $("#links li").find("a[href='"+loc+"']").each(function(){ 
     $(this).addClass("active"); 
    }); 
}else(typeof loc === 'undefined'){ 
    $("#links li a:eq(0)").addClass("active"); // if you use 'home' as your first. 
} 

Answer 3

E 'probabilmente che la classe attiva deve essere applicato il li .. Supponendo che la tua logica è corretta e la classe deve essere applicato il Li (difficile dire senza l'HTML e CSS):

if (pathname.indexOf(href) >= 0){ 
     $(this).parent().addClass("active"); 
} 

Answer 4

io non sono sicuro di ottenere la questione a tutti, ma Vorrei andare per qualcosa del tipo:

$("#links ul li a").filter(function() { 
    return window.location.pathname.indexOf(this.href) != -1; 
}).addClass('active'); 

Answer 5

<script type="text/javascript"> 

$(window).load(function() { 
        var url = window.location; 
        $('#nav a').filter(function() { 
         return this.href == url; 
        }).parents("li").addClass('active'); 

      }); 
</script> 

è un lavoro uso questo script quando si utilizza file esterno per il menu

come index.php inclide menu.php è necessario utilizzare questo scritp tu rilevare url caricato e menù di classe modifica alla classe di attivi.

Grazie U thecodeparadox il tuo codice è lavoro