basic javascript accordion

Question

Я делаю аккордеон FAQ. Есть много сценариев в Интернете, но я ненавижу использовать сценарии, которые я не понимаю (я новичок в Javascript). Поэтому я начинаю с базового и работаю. Следующий код расстраивает меня, почему он не работает:

Javascript:

function faq1() { 

var faq1 = document.getElementById("faq1") 

if (faq1.style.display = "none") { 
faq1.style.display = "block"; 
} 

else { 
faq1.style.display = "none"; 
} 
} 

HTML/CSS:

<a href="#" onclick="faq1()">Question 1</a> 
<div id="faq1" style="display: none"> 
Answer to question 1 
</div> 

скрипт открывает аккордеон просто найти. Тем не менее, «else» часть Javascript не работает, когда я нажимаю на ссылку во второй раз, я не могу заставить дисплей отключиться. Я пробовал различные методы «else if», чтобы увидеть, будет ли это работать без везения.

Спасибо ребята

Answer 1

Вы используете назначение (=) вместо сравнения (==) в вашем состоянии.

В качестве альтернативы,

function faq1() { 

    var faq1 = document.getElementById("faq1"); 

    faq1.style.display = faq1.style.display == "none" ? "" : "none"; 

}