La struttura del JS e` corretta.
La struttura HTML e` completamente da rivedere secondo le moderne tecniche (quella struttura e` praticamente HTML2, e potrebbe dare qualche problema, sopratutto nel posizionamento).
Poi io preferirei assegnare due CSS all'elemento, esempio:
codice:
function inizial(showHideDiv) {
var visibile = 1; // di default visibile
var ele = document.getElementById(showHideDiv);
}
function toggle5(showHideDiv) {
if(visibile) {
ele.style.className = "clHidden";
visibile = 0;
} else {
ele.style.display = "clVisible";
visibile = 1;
}
}
dove clVisible e clHidden sono le classi da assegnare al blocco per renderlo visibile o nascosto
e chiaramente inzial() e` da far girare all'evento onload, mentre il toggle e` richiamato dal bottone.
PS: quando posti codice, usa i tag appropriati (ad esempio CODE)