Что не так в Javascript?

Question

[Михаил Гильмутдинов]

if ($(window).width() > 991) {
document.querySelector("#nav-toggle").addEventListener("click", function() {
	this.classList.toggle("active");
});
}

Uncaught TypeError: Cannot read property 'addEventListener' of null

Answer 1

[Алексей Уколов]

document.querySelector("#nav-toggle")
На странице нет такого элемента, поэтому querySelector возваращает null.
Об этом в ошибке и написано - у null нет метода addEventListener.
addEventListener вызывается только в одном месте - правда, разобраться совсем не сложно.

А вообще, если уж используете jquery, так и используйте на полную катушку:

if ($(window).width() > 991) {
    $("#nav-toggle").on("click", function() {
        $(this).toggleClass("active");
    });
}

Comments

[Михаил Гильмутдинов]

как исправить?

[Алексей Уколов]

Может, попробовать добавить элемент с id "nav-toggle" на страницу?

[Алексей Уколов]

Михаил Гильмутдинов: если хотите получить более развернутый ответ, покажите демо на jsfiddle.

Answer 2

[Дима nahkar]

Вот решение

if (window.innerWidth > 991) {
    document.querySelector("#nav-toggle").addEventListener("click", function(e) {
         e.target.classList.toggle("active");
    },false);
}

Comments

[Алексей Уколов]

Решение чего? Откуда взялся селектор .logo? Зачем вместо this обращаться к элементу через e.target?