Problema Onclick

[fattobene]

Ragazzi praticamente devo far si che il link menu rimanga cliccato rispetto agli altri una volta che lo "clicco", e cosi vale per gli altri

ecco ciò che ho scritto:

function selection(val){
if(val==1)
document.getElementById("menu").style.className="l inkMenuHome"
else
document.getElementById("menu").style.className="l inkMenu"
if(val==2)
document.getElementById("menu").style.className="l inkMenuHome"
else
document.getElementById("menu").style.className="l inkMenu"
if(val==3)
document.getElementById("menu").style.className="l inkMenuHome"
else
document.getElementById("menu").style.className="l inkMenu"
if(val==4)
document.getElementById("menu").style.className="l inkMenuHome"
else
document.getElementById("menu").style.className="l inkMenu"
}

e nell'html della pagine il link presenta questo codice:

<a id="menu" onclick="selection('1');" class="linkMenu">HomePage</a>

<a id="menu" onclick="selection('2');class="linkMenu" >Chi Siamo</a>

...
..
..
<a id="menu" onclick="selection('4');class="linkMenu" >Contatti</a>

Cosa Ho sbagliato che non mi rimane selezionato il Link Menu?

Grazie

[br1]

Devi chiudere i gruppi if - else con un bel ;

Meglio ancora usa le parentesi:

if (condizione) { istruzioni1 } else { istruzioni2 }

ciao

[fattobene]

Originariamente inviato da br1
Devi chiudere i gruppi if - else con un bel ;

Meglio ancora usa le parentesi:

if (condizione) { istruzioni1 } else { istruzioni2 }

ciao

Anche le parentesi non hanno risolto il tutto....

[br1]

Non avevo notato il link

Contatti

Tra l'altro la funzione si aspetta un numero... e tu le passavi una stringa

ciao

Edit: inoltre tutti gli else tranne uno verranno eseguiti... scrivile cosi':

function selection(val){
document.getElementById("menu").style.className="l inkMenu";
if(val==1)
document.getElementById("menu").style.className="l inkMenuHome";
if(val==2)
document.getElementById("menu").style.className="l inkMenuHome";
if(val==3)
document.getElementById("menu").style.className="l inkMenuHome";
if(val==4)
document.getElementById("menu").style.className="l inkMenuHome";
}

ciao

[fattobene]

Originariamente inviato da br1
Non avevo notato il link

Contatti

Tra l'altro la funzione si aspetta un numero... e tu le passavi una stringa

ciao

Edit: inoltre tutti gli else tranne uno verranno eseguiti... scrivile cosi':

function selection(val){
document.getElementById("menu").style.className="l inkMenu";
if(val==1)
document.getElementById("menu").style.className="l inkMenuHome";
if(val==2)
document.getElementById("menu").style.className="l inkMenuHome";
if(val==3)
document.getElementById("menu").style.className="l inkMenuHome";
if(val==4)
document.getElementById("menu").style.className="l inkMenuHome";
}

ciao

Non funziona anche correggendo il tutto!!!!! :berto:

[br1]

Posta tutta la pagina...

ciao

Edit: aspetta, a quanto pare hai scelto un nome infelice per la funzione: selection non piace ad IE... se la chiami pippo tutto funziona VVoVe: