2017-09-13 35 views
0

親ヘッダーのonclickイベントで表示および非表示になるニュースレターサムネイルのリストを作成しようとしています。この関数は、リストを表示するために適切に呼び出されますが、2回目のクリックでそれを再び隠すことはありません。私はそれが私が行方不明のシンプルなものだと確信していますが、私は迷っています。親のonclickイベントで非表示divを表示/非表示

function showBabList() { 
 
    if (document.getElementById('bablist').style.display = "none") { 
 
    document.getElementById('bablist').style.display = "block"; 
 
    } else { 
 
    document.getElementById('bablist').style.display = "none"; 
 
    } 
 
}
<h2 onclick="showBabList()">BITSandBYTES Newsletters</h2> 
 
<div id="bablist" style="display: none;"> 
 
    <ul> 
 
    <a href="newsletter/BITSandBYTES2017-07.pdf" target="bab"><img src="images/thumbs/BITSandBYTES2017-07_thumb.png" class="babthumb" height="100px" width="75px"></a> 
 
    <a href="newsletter/BITSandBYTES2017-08.pdf" target="bab"><img src="images/thumbs/BITSandBYTES2017-08_thumb.png" class="babthumb" height="100px" width="75px"></a> 
 
    </ul> 
 
</div>

+0

私はあなたの代わりに=のdocument.getElementById( 'bablist')の==を使用する必要があると思う。style.display == "なし" –

答えて

1

function showBabList() { 
 
    if (document.getElementById('bablist').style.display == "none") { 
 
    document.getElementById('bablist').style.display = "block"; 
 
    } else { 
 
    document.getElementById('bablist').style.display = "none"; 
 
    } 
 
}
<h2 onclick="showBabList()">BITSandBYTES Newsletters</h2> 
 
<div id="bablist" style="display: none;"> 
 
    <ul> 
 
    <a href="newsletter/BITSandBYTES2017-07.pdf" target="bab"><img src="images/thumbs/BITSandBYTES2017-07_thumb.png" class="babthumb" height="100px" width="75px"></a> 
 
    <a href="newsletter/BITSandBYTES2017-08.pdf" target="bab"><img src="images/thumbs/BITSandBYTES2017-08_thumb.png" class="babthumb" height="100px" width="75px"></a> 
 
    </ul> 
 
</div>

WRONG
if (document.getElementById('bablist').style.display = "none")

CORRECT GEORGE BY
if (document.getElementById('bablist').style.display == "none")

+0

、あなたは正しいです。私は、関数が最初に呼び出されたので、それをさらに解決する必要はないと考えました。 – RevFonzAyy

関連する問題