Was ist daran falsch?

Tobsen

Was ist an folgendem Codeausschnitt falsch?

function close_menu(untermenu_id, sec_untermenu_id)
     {
      if(document.getElementById(sec_untermenu_id).style.visibility=="visible")
       {
         document.getElementById(untermenu_id).style.visibility='visible';
       }
      else{document.getElementById(untermenu_id).style.visibility='hidden';}
     }

MfG

Tobsen

ms

Meinst du mit Falsch die Logik oder die Syntax?
Ich kann nur erkennen, dass du im ersten if "visible" und die anderen male 'visible' bzw. 'hidden' hast.
Keine Ahnung ob das jetzt dein Fehler ist.

der fehler ist der:
das du überprüft ob das sec_untermenue_id sichtbar ist
und wenn es sichtbar ist wird er sichtbar gemacht

richtig müsste es sein:
wenn es sichtbar ist muss es unsichtbar gemacht werden :p

also

function close_menu(untermenu_id, sec_untermenu_id)
{
      if(document.getElementById(sec_untermenu_id).style.visibility=="visible")
      {
         document.getElementById(untermenu_id).style.visibility='hidden';
      }
      else
      {
         document.getElementById(untermenu_id).style.visibility='visible';
      }
}

mfg