Ukrywanie diva Opcje

[Spyder]

Witam mam taki skrypt:

[HTML] pobierz, plaintext 
var state = 'none';
function showhide(layer_ref){
if(state == 'block'){
state = 'none';
}else{
state = 'block';
}
if(document.all){ //IS IE 4 or 5 (or 6 beta)
eval("document.all." +layer_ref+ ".style.display = state");
}
if (document.layers) { //IS NETSCAPE 4 or below
document.layers[layer_ref].display = state;
}
if (document.getElementById &&!document.all) {
hza = document.getElementById(layer_ref);
hza.style.display = state;
}
}
[HTML] pobierz, plaintext 

[HTML] pobierz, plaintext 
<ul>
<li><span onclick="showhide('cat2');">Exempt</span>
<ul id="cat2" style="display:none" >
<li><a href="ex-pending.php" title="Pending Exempt">Pending Exempt</a></li>
<li><a href="ex-completed.php" title="Completed Exempt">Completed Exempt</a></li>    
</ul>
</li>
</ul>
[HTML] pobierz, plaintext 

I teraz jak klikne w Exempt to otworzy mi ul. Wszystko dziala tyle ze czasami, szczegolnie na poczatku musze kliknac 2x zeby div sie otworzyc. Jak to przerobic zeby status bylk ladowany na onload. Dzieki

[wookieb]

Aaaa. A ja już myślałem ze masz jeden taki element.
Logicznie rzecz myśląc masz jeden state dla wszystkich elementów dlatego teraz widzisz co jest nie tak

Kod

function showhide(layer_ref)
{
var lay=document.getElementById(layer_ref);

if(!lay) return false;

if(lay.style.display=='' || lay.style.display='none') lay.style.display='block';
else lay.style.display='none';

}

Możesz też zmieniac im klasy.