Проблема в Javascript

[povelitel]

Есть конструкция html

<ul id="nav" onclick="myFun('nav')">
    <li onclick="myFun('myLi1')"><a href="#">HOME</a>
        <ul id="myLi1">
          <li><a href="#">HOME</a></li>
          <li><a href="#">PORTFOLIO</a></li>
          <li><a href="#">BLOG</a></li>
        </ul>
    </li>
    <li onclick="myFun('myLi2')"><a href="#">PORTFOLIO</a>
        <ul id="myLi2">
          <li><a href="#">HOME</a></li>
          <li><a href="#">PORTFOLIO</a></li>
          <li><a href="#">BLOG</a></li>
        </ul>
    
    </li>
        <li><a href="#">BLOG</a></li>
        <li><a href="#">SERVICES</a></li>
        <li><a href="#">CASE STUDIES</a></li>
        <li><a href="#">CONTACTUS</a></li>
    </ul>

и есть сам скрипт, который при клике открывает спрятаные ссылки в теге ul.

function myFun(n){

var myBlock = document.getElementById(n);

if(myBlock.style.display = 'none'){

myBlock.style.display ='block';

}
}

вопрос: Как сделать так, чтобы при повторном клике список ul закрывался обратно?

[Stilet]

if(myBlock.style.display = 'none'){
myBlock.style.display ='block';
} else {
myBlock.style.display ='none';
}

[povelitel]

Сообщение от Stilet

if(myBlock.style.display = 'none'){
myBlock.style.display ='block';
} else {
myBlock.style.display ='none';
}

Такой вариант не работает

[nerv]

if( myBlock.style.display === 'block' || !myBlock.style.display ) {
    myBlock.style.display = 'none';
} else {
    myBlock.style.display = 'block';
}

че т голова туго варит ^__^

[Stilet]

Такой вариант не работает

Потому что в JS оператор сравнения это ==.

[Кольша]

<ul id="nav" onclick="myFun(this)">
.....
function myFun(n){
n.style.display=(n.style.display=='block')? 'none': 'block';}

Кто напишет код меньше?))

[Stilet]

Например я:

<ul id="nav" onclick="this.style.display=(this.style.display=='block')? 'none': 'block';}">

[Кольша]

Ну если можно было убрать функцию..то воот:

<ul onclick="this.style.display=(this.style.display=='block')?'none':'block';">

на пару символов меньше))

[MyLastHit]

Какой же это свитчер? После display:none по нему кликнуть уже не получится)

[Stilet]

MyLastHit

А ты прав. Без функции тут никак ИМХО.
Хотя...

    <li onclick="this.children[1].style.display=(this.children[1].style.display=='block')?'none':'block';"><a href="#">HOME</a>
        <ul id="myLi1">
          <li><a href="#">HOME</a></li>
          <li><a href="#">PORTFOLIO</a></li>
          <li><a href="#">BLOG</a></li>
        </ul>
    </li>

не?