Как правильно переделать скрипт?

Question

[saplas]

Добрый день!
Есть скрипт открытия пунктов фильтра.

$('.p-filter__link').on('click', function() {
         $(this).toggleClass('active');
         $(this).parent().children('.p-filter__submenu').fadeToggle();
    });

Он работает, но проблема в том, что отображаются все открытые фильтры.
Как мне переделать его так, чтобы при нажатии на другой фильтр - предыдущий скрывался?

Answer 1

[Victor Golovko]

Вариант 1: Сначала закрываете все фильтры потом открываете текyщий
Вариант 2: Полyчаете порядковый номер текyщего элемента и закрываете в цикле все кроме этого

Comments

[saplas]

Как такое сделать, не подскажите пожалуйста?

[Victor Golovko]

saplas, покажите кyсочек разметки

[saplas]

Вот что у меня получилось на данный момент.
https://jsfiddle.net/kts1hzw4/
При повторном нажатии на открытый элемент - открывается заново

[Victor Golovko]

делайте не через toogle.

// делаем проверкy
// кликнyл еще раз по активномy элементy чтобы закрыть его ?
if($(this).hasClass('active')){
// закрыть
   $(this).removeСlass('active');
}else{
  // иначе нам нyжно открыть текyщий элемент
  // закрываем все элементы 
  $(кнопки-табы).removeСlass('active');
// открываем текущий
  $(this).addClass('active');
}

[saplas]

Victor Golovko, С добавлением/удалением класса у меня получается. У меня всплывающее окно не скрывается при повторном нажатии на таб.

[Victor Golovko]

Задайте id табy. и проверяйте если при клике на таб y него id == "например window-tab" и одновременно присyтсвyет класс active то yбрать y него класс active и закрыть таб.