[FRAGE] JS über Button visibility wechseln

[MrPaule75]

Hallo Community,
ich suche und versuche nun schon eine Weile und erlaube mir, mich an Euch zu wenden.

Ich erstelle eine Webseite, auf der das Navigationsmenü in einem versteckten div liegt und über einen Button aufgerufen wird. Es handelt sich dabei um ein animierten Button, welcher auf Klick den Layer einblendet und das Symbol von "=" auf "X" wechselt. Von "X" zu "=" im Button gelange ich nur, wenn ich wieder auf diesen Button klicke. Muss also ein weiteres Mal genutzt werden, damit sich die Symbolik ändert. Demzufolge muss ich über den selben Button den Layer wieder von "visible" auf "hide" wechseln.

<script>
<!--
    function toggle_visibility(id) {
       var e = document.getElementById(id);
       if(e.style.visibility == 'visible')
          e.style.visibility = 'hidden';
       else
          e.style.visibility = 'visible';
    }
//-->
</script>

<!-- Button zum ein- und ausblenden des Menülayer -->
<a onClick="document.getElementById('menue-layer-hidden').style.visibility= toggle_visibility();">
<button class="c-hamburger c-hamburger--htx">

<!-- Menülayer -->
<nav>
<div id="menue-layer-hidden" class="menue-klasse">

Der Button soll beim Laden der Seite quasi den Menülayer beim Klick einblenden:

<a onClick="document.getElementById('menue-layer-hidden').style.visibility= 'visible'">

Ist der Layer dann aktiv, müsste beim Klick auf den Button ausgeführt werden:

<a onClick="document.getElementById('menue-layer-hidden').style.visibility= 'hidden'">

Bei meinen Versuchen reagiert zwar der Button (Animation) aber der Menülayer wird gar nicht eingeblendet. Ist der Code falsch oder habe ich einen Denkfehler?

Vielen Dank im Voraus
Grüße
MrPaule

 

[Daniel Pa]

Du hast vergessen, bei toggle_visibility einen Parameter zu uebergeben:

<a onClick="document.getElementById('menue-layer-hidden').style.visibility= toggle_visibility('menue-layer-hidden');">

 

[MrPaule75]

Hallo Daniel,

boa ... super. Hatte ich vor lauter Bäumen nicht bemerkt.
Herzlichen Dank!

 

[kkapsner]

Da toggle_visibility() aber keinen Rückgabewert hat (bzw. undefined), ergibt es keinen Sinn, diesen an irgendwas zuzuweisen.

<a onclick="toggle_visibility('menue-layer-hidden');">

PS: Dein Problem hättest du ev. mit Hilfe der Fehlerkonsole selbst gefunden.
PPS: es heißt onclick...

 

[mikdoe]

PPS: es heißt onclick...

ich meine kürzlich gesehen zu haben, dass das bei inline js offiziell nicht zwingend ist.

 

[kkapsner]

Das ist in HTML nicht zwingend, da HTML case-insensitiv ist.
JS ist immer (auch inline) case-sensitiv.

PS: In den HTML5 Specs wird es klein geschrieben: 7 Web application APIs | HTML 5.1