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

Question

[DenKuzmin17]

не работает часть:

var current = this.slider.parent().parent().parent().find('.active').eq(this.currentImg);
            current.removeClass('active');
            this.slider.parent().parent().parent().find('.set').eq(this.currentImg).addClass('active');

Идея такая, что при scrollImage у элемента .active убираем класс active и присваиваем .active элементу с индексом равном новому currentImg. Активный класс "переходит", но так же активным оказывается прошлый элемент.

Помогите, где ошибка?

Comments

[ilyamst51]

Просто замечание Вам.
Если вам нужно найти родителя элемента дальше, чем в 1 узел, используйте parents(). Не обязательно делать parent(). Поверьте мне. Просто вбейте в яндекс jquery parents, там будет все описано

[DenKuzmin17]

ilyamst51, благодарю. Поправил.

Answer 1

[ilyamst51]

Вместо

var current = this.slider.parent().parent().parent().find('.active').eq(this.currentImg);
current.removeClass('active');

Напишите

var current = this.slider.parent().parent().parent().find('.active');
current.removeClass('active');

И все.

Comments

[ilyamst51]

Объясняю почему. Вот вы нашли элемент с классом active. Вы уже его имеете. Ну то есть все, он ваш. Зачем .eq ?

[ilyamst51]

Ну и конечно не забудьте избавиться от parent() 3 раза в ряд

[DenKuzmin17]

Спасибо большое!

[DenKuzmin17]

ilyamst51, eq() от старого скрипта осталось. Не заметил. Спасибо большое.