Почему не работает конструкция if else?

Question

[AnnaUniq]

Вот ссылка на пример dev.sholex.ru/acc.html
Необходимо, чтобы при клике на элемент li ему же присваивался класс active, но при клике на другой li - удалялся
В примере имеется аккордеон, где предпоследний пункт меню (о компании) имеет двойную вложенность, так вот как реализовать, чтобы при клике на пункт меню Вакансии его родительский li(о компании) не терял класс active
пробую конструкцией if else на проверку display block у дочернего элемента ul, но как-то не выходит

if ($('.sub-menu').css('display') !== 'block'){
    		$(this).addClass('active');  
		}
		else{
			$('li.menu-item-has-children').removeClass('active');
		};

может кто-то другое решение подскажет
спасибо!

Comments

[iDrugov]

!= а не !==

есть сомнения по поводу всей конструкции, но начни со знака

[AnnaUniq]

iDrugov: проблема заключается лишь в том, чтобы при открытии 3 уровня в акке, сохранялся класс у первого открытого

Answer 1

[Андрей]

Я бы предложил такой вариант:

1. При клике запомнить элемент, на который кликнули
2. Сбросить класс `active` у всех элементов меню
3. Проставить класс `active` для сохраненного элемента
4. В цикле проставить `active` для его родителей, пока не выйдем за пределы меню

Answer 2

[Игорь Перегудов]

$('.sub-menu').css('display') - так задается css для блока "$('.sub-menu').css('display': 'block)

вы делайте проще,

$('li').on('click', function () {
    if($(this).hasClass('active')){
        $(this).removeClass('active');
    }else {
        $(this).addClass('active');
        $(this).siblings().removeClass('active');
    }
});

Comments

[AnnaUniq]

только вот если повторно кликнуть на тот же элемент класс не исчезает, проблема с вложенностью осталась, т.е. если открываем 3 уровень меню, то у его родительского li класс исчезает

[Игорь Перегудов]

AnnaUniq: отредактировал ответ

[AnnaUniq]

Игорь Перегудов: все равно не работает,буду разбираться, спасибо!

[Игорь Перегудов]

AnnaUniq: я вам показал реализацию, если вы бездумно вставляете код - не мудрено, что не будет работать