рони,
Так они вообще не должны по идее присутствовать несколько сразу. Только 1 элемент
li с классом
active должен быть.
activeTab = document.getElementsByClassName("tab active");
Определяю его в переменной activeTab
При клике на кнопку проверяю, есть ли такой элемент уже.
if(activeTab !== null) {
activeTab.classList.remove("active")
}
Не равен нулю, значит, есть. Значит, удаляем у него класс
active.
По этому раскладу в принципе не должна идти проверка среди нескольких элементов, такой элемент всегда один.
А по факту получается, что функция игнорирует это и класс
active вообще не удаляется. Хотя сам элемент через console.log определяется.