Vermeer: Ich finde den Fehler in diesen paar Codezeilen nicht.

Servus,
kann mir jemand sage wo der Fehler liegt?
Einblenden funktioniert, aber ausblenden nicht:

<html>
<head>
<title>Untitled</title>
<script type="text/javascript">
function changevisibility()
{
  if (document.getElementById("editor").style.display  = 'none')
  {
   document.getElementById("editor").style.display  = 'block';
   }
  else
  {
   document.getElementById("editor").style.display = 'none';
  }
}
</script>

</head>

<body>

<a href="javascript:changevisibility();">ein und ausblenden</a>

<div id="editor" style="display:none;">
Blabalbuberblub
</div>

</body>
</html>

  1. Hi,

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

    Du möchtest doch vergleichen ;)

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

    Gruß,
    Felix

    --
    Nichts auf der Welt ist so gerecht verteilt wie der Verstand. Denn jedermann ist überzeugt, dass er genug davon habe.
    René Descartes
  2. Yerf!

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

    Das hier sollte wohl besser ein Vergleich und keine Zuweisung sein, also mit == und nicht mit nur einem.

    Gruß,

    Harlequin

    --
    <!--[if IE]>This page is best viewed with a webbrowser. Get one today!<![endif]-->
  3. Servus,

    Hallo,

    kann mir jemand sage wo der Fehler liegt?

    Du brauchst in deiner if-Bedingung keinen Zuweisungs- sondern einen Vergleichsoperator. Sonst wäre das Ergebnis in diesem Fall immer true.

    Gruß, Volker

  4. Okay, danke an alle für die schnelle Antwort, das war genau der Fehler.
    Das lerne ich wohl nie mit dem "=" und "==" :)

    Vermeer