Element wird nicht versteckt

[Folon]

Hallo erstma^^

ich komme gleich mal zu meinem Problem:
Ich will, dass man mit 2 Buttons 2 Elemente (also je ein Button für ein Element meine ich damit) ein/ausblenden kann (eine Uhr und ein Logo).
Mit dem "Drücken" auf einen Button (Bsp. den für die Uhr) wird die Funktion sichtbarkeit('uhr') aufgerufen.
Ich glaube, ich poste am besten mal mein Javascript^^

function sichtbarkeit( elementt )
{
   if ( elementt == "uhr" )
   {
      alert( "Uhr" );
      if ( uhrsichtbar === true )
      {
         alert("Uhr ist sichtbar!");
         uhrsichtbar = false;
         document.getElementById( "time" ).style.display = none;
         document.getElementById( "uhrbutton" ).value = "Uhrzeit anzeigen";
      }
      else
      {
         alert("Uhr ist versteckt!");
         uhrsichtbar = true;
         document.getElementById( "time" ).style.display = inline;
         document.getElementById( "uhrbutton" ).value = "Uhrzeit verbergen";

      }
   }
   else
   {
      alert( "Logo" );
      if ( logosichtbar === true )
      {
         alert("Logo ist sichtbar!");
         logosichtbar = false;
         document.getElementById( "headdd" ).style.display = none;
         document.getElementById( "logobutton" ).value = "Logo zeigen";
      }
      else
      {
         alert("Logo ist versteckt!");
         logosichtbar = true;
         document.getElementById( "headdd" ).style.display = inline;
         document.getElementById( "logobutton" ).value = "Logo verstecken";

      }

   }
}

Und die Buttons zum aufrufen

<input type="button" name="logobutton" onClick="sichtbarkeit('logo');" id="logobutton" value="Logo verstecken" />|<input type="button" name="uhrbutton" onClick="sichtbarkeit('uhr');" id="uhrbutton" value="Uhrzeit verbergen" />

Was ich noch erwähnen sollte ist, dass die Funktion aufgerufen wird ich habe schon alle Alert-Nachrichten bekommen nur wird weder der Text der Buttons geändert noch werden die Elemente versteckt, also denke ich, dass der Fehler darin liegt.

 

[dasch]

Mach es mal so:

function sichtbarkeit( elementt )
{
   if ( elementt == "uhr" )
   {
      alert( "Uhr" );
      if ( uhrsichtbar === true )
      {
         alert("Uhr ist sichtbar!");
         uhrsichtbar = false;
         document.getElementById( "time" ).style.display = 'none';
         document.getElementById( "uhrbutton" ).value = "Uhrzeit anzeigen";
      }
      else
      {
         alert("Uhr ist versteckt!");
         uhrsichtbar = true;
         document.getElementById( "time" ).style.display = '';
         document.getElementById( "uhrbutton" ).value = "Uhrzeit verbergen";

      }
   }
   else
   {
      alert( "Logo" );
      if ( logosichtbar === true )
      {
         alert("Logo ist sichtbar!");
         logosichtbar = false;
         document.getElementById( "headdd" ).style.display = 'none';
         document.getElementById( "logobutton" ).value = "Logo zeigen";
      }
      else
      {
         alert("Logo ist versteckt!");
         logosichtbar = true;
         document.getElementById( "headdd" ).style.display = '';
         document.getElementById( "logobutton" ).value = "Logo verstecken";

      }

   }
}

Du hast die anführungsstriche vergessen. Und wenn du eine Div ausblenden willst nimm '' anstatt 'inline'

 

[Folon]

und schon Funktioniert es...

danke dasch

 

[crash]

nimm lieber className, als den Stil direkt zu verändern. Lässt sich einfacher warten.