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

Question

[Crushing]

function AdaptiveMenu() {
	var Adaptive = document.getElementById('adaptiveNavbar');
	var AdaptiveStyle = getComputedStyle(Adaptive);
	if (AdaptiveStyle.display = 'none') {
		AdaptiveStyle.display = 'block';
	};
}

При клике на иконку, должна вызываться функция которая смотрит, если у элемента в свойствах display:none, то сделать его display:block, и наоборот.

Такая ошибка:

Comments

[Сергей delphinpro]

ну может быть потому что опечатка
= / ==

[Anton Mashletov]

Писать название переменный и функций с большой буквы - дурной тон.

[Crushing]

Сергей, Опечатку исправил, теперь код просто не работает, без ошибки всякой

[Сергей delphinpro]

Crushing, https://jsfiddle.net/yxsj1qhd/

[ой всё]

браузер при отладке показывает значение переменных на каждом шаге
что вам мешает самостоятельно посмотреть, что и где и у вас не работает?

Answer 1

[mrxakerrus]

1) В операторе if, вы пытаетесь ПРИСВОИТЬ ЗНАЧЕНИЕ, А НЕ СРАВНИТЬ
2) Вызывать свойства у элемента нужно через GetComputedStyle либо через style

Comments

[Crushing]

Присвоить значение убрал, Так и вызываю свойства.

[mrxakerrus]

Ааааа все, понял..
if(Adaptive.style.display==""){
Adaptive.style.display="block";
}

[mrxakerrus]

Тоже кстати не совсем верно, вот верный вариант
If(getComputedStyle(Adaptive).display =="none"){
Adaptive.style.display="block";
}

[Crushing]

https://jsfiddle.net/Laa3yqby/5/ - Вот на JSFiddle

[mrxakerrus]

Crushing, 1)изучи вызов функции при событии onclick="adaptiveMenu()"
2) присваивание и сравнение (ошибка внутри блока теперь ты пытаешься сравнить уже, а не присвоить)
3) Используя getComputedStyle метод, он как бы намекает на то что он получает свойства, а не присваивает, а присваивать нужно напрямую через [id].style.display="block";

function AdaptiveMenu() {
var adaptive = document.getElementById("menu");
if (getComputedStyle(adaptive).display == "none") {
console.log("STYLE");
adaptive.style.display = 'block';
};
}