Как правильно менять активный класс?
Как может меняться активный класс, если вы нигде его не меняете?
snippet
убери класс активности в верстке и добавляй/убирай его динамически, тогда все будет ок

или же можешь оставить его и пользоваться методом toggle()