JS добавление active

Немного подробней опишите что именно вы хотите. И что не так получаете сейчас.

Вопрос элементарный, но вы как то непонятно описали...

Если я правильно понял:

У вас меню со ссылками и при опред. условии вы хотите одному из пунктов меню добавить класс "active".

Судя по js коду: Класс active добавляется обертке ссылки которая ведет на текущую страницу.

А теперь, напишите что вы хотите. В идеале со ссылкой на сайт.

https://pixel-mosaic.ru/2019/01/15/galereja-dlja-vannoj/

Active сейчас добавляется для класса "projects-tab"

Я последний пункт меню специально не трогал, чтобы было с чем сравнить.

А предыдущие пункты я обернул в <a href..> и присвоил новый класс "projects-a" - именно этому классу сейчас нужно добавлять active, чтобы было "projects-a active".

Наверное, так

function addActiveLinkNavigation(){
    var siteUrl = window.location.href;
    $('a.projects-a').each(function(){
      if($(this).attr('href')==siteUrl){$(this).addClass('active');}
    });
}

поиграем в милторга

function addActiveLinkNavigation(){

     $('a.projects-a[href="'+window.location.href+'"]').addClass('active');

}

function addActiveLinkNavigation(){
    var siteUrl = window.location.href;
    $('a.projects-a').each(function(){
      if($(this).attr('href')==siteUrl){$(this).addClass('active');}
    });
}

Большое спасибо! Заработало

---------- Добавлено 25.02.2019 в 22:09 ----------

function addActiveLinkNavigation(){
     $('a.projects-a[href="'+window.location.href+'"]').addClass('active');
}

Как оказалось active нужен для "projects-tab", т.к. только в нем можно залить правильно background.

Перепробовал разные варианты, ничего не срабатывает:

function addActiveLinkNavigation(){
     $('a.projects-tab[href="'+window.location.href+'"]').addClass('active');
}

function addActiveLinkNavigation(){
     $('.projects-tab[href="'+window.location.href+'"]').addClass('active');
}

function addActiveLinkNavigation(){
     $('.projects__nav .projects-tab[href="'+window.location.href+'"]').addClass('active');
}

Изначальный вариант тоже не работает, т.к. я div поместил внутрь <a href..>

Stasmotion, тогда надо было сделать вот так:

function addActiveLinkNavigation(){
     $('a.projects-a[href="' + window.location.href + '"] > .projects-tab ').addClass('active');
}

этот пример хорош тем, что не использует перебор в цикле (вернее использует но на уровне парсера dom а не на уровне js интерпретатора), который достаточно медленный, поэтому, при некоторых условиях, подобная функция будет работать гораздо быстрее. Может быть даже в разы, чем вариант с each

зы

Ненадо помещать блочный элемент внутрь строчного, это не очень хорошая практика (скажем так, одно время оно даже считалось ошибкой)

А вот в этом я ой как не уверен. DOM выстраивается с учётом классов, а здесь используется значение атрибута - и мне сомнительно, что этот цикл не исполняется именно на уровне JS.

Все эти глупости имеют смысл больше для того, чтобы минифицировать код в стиле милторговских олимпиад, а по производительности - "бабка надвое сказала". Jquery - это библиотека, и надо смотреть, какие циклы там исполняются, чтобы докопаться до корней. Попробуйте сделать аналогичную операцию средствами чистого JS - и всё это безобразие вылезет как есть.

---------- Добавлено 26.02.2019 в 12:20 ----------

Sitealert, Ну сделать эту операцию чистым джи эс это тоже не вопрос сделать.

 document.querySelector('a.projects-a[href="' + window.location.href + '"] > .projects-tab ').classList.add('active'); 

как видите, большой разницы в синтаксисе нет.

хотя, может быть вы и правы по поводу jquery

есть вероятность, что там дергается querySelectorAll, а потом циклом обегается.

Если active добавлять для "projects-a" то кнопка потом вот так заливается, вылезает за края http://prntscr.com/mqanq2

А если для "projects-tab" то все ок http://prntscr.com/mqaoz7

Но как добавлять в текущем JS active именно для "projects-tab" я так и не понял.

Немного почитал на эту тему и пожалуй соглашусь с Вами. Внутри Jquery при наличии валидного селектора используется именно метод querySelector.