Почему код не работает?

Question

[agent1156]

$('.menu-active li ').each(function () {
        if($(this).attr('href') == location.href) { $(this).addClass('active')};
        
    });

Answer 1

[Ivan Bogachev]

В таких случаях codepen/jsfiddle очень приветствуется. Пока его нет, вангую, что вы пытаетесь достать аттрибут href не из ссылки, а из элемента списка, у которого href отсутствует.

Comments

[agent1156]

как узнать url текущей страницы ?

[Ankhena]

agent1156: location.pathname или window.location.pathname
адрес страницы без домена

Answer 2

[justdealman]

Попробуйте
location.pathname
вместо
location.href

Answer 3

[Артем Можаров]

потому что надо

$('.menu-active li <b>a</b> ').each(function () {
     if($(this).attr('href') == location.href) { $(this).addClass('active')};
 });

Так как у li нет атрибута href