Discussion :

[laurent2]

bonjour tout le monde!
voici mon problème:
j'ai deux boutons radio, et le premier est "checked" par défaut et ne change en rien l'apparence de mon formulaire.
je voudrais faire apparaître un DIV lorsque je clique sur le deuxième...
j'ai fait un essais avec le code suivant, mais évidement, cela ne fonctionne pas...
pouvez-vous me dire où je me suis planté...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
<!--code de le HEAD--!>
function afficher(reservation)
{
	document.getElementById("reservation").style.visibility = "";
}
function cacher(reservation)
{
	document.getElementById("reservation").style.visibility = "hidden";
}
 
<!--code des boutons radio--!>
        <p id="Pnormal3"><input type="radio" name="motif" value="Renseignements" checked="checked" onClick="cacher(reservation)" />Renseignements</p>
        <p id="Pnormal3"><input type="radio" name="motif" value="Reservations" onClick="afficher(reservation)" />R&eacute;servation</p>
 
<!--DIV à afficher--!>
 <div id="reservation" style="display: none">
 </div>

merci d'avance!

[buzzkaido]

Pas de quotes dans getElementById : c'est la variable que tu utilises

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
function afficher(reservation)
{
	document.getElementById(reservation).style.visibility = "";
}
function cacher(reservation)
{
	document.getElementById(reservation).style.visibility = "hidden";
}

Des quotes ici car c'est une chaine que tu passes en parametre et un point-virgule à la fin, c'est plus propre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

onClick='afficher("reservation");'

Pour eviter de confondre parametre, chaine et variable, tu devrais plutot l'ecrire comme ça :
(c'est la même chose, y'a juste les noms de changé)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
function afficher(elemID)
{
	document.getElementById(elemID).style.visibility = "";
}
function cacher(elemID)
{
	document.getElementById(elemID).style.visibility = "hidden";
}

elemID pour "element ID" histoire d'être cohérent avec l'utilisation faite de ce parametre

[laurent2]

Pas de quotes dans getElementById : c'est la variable que tu utilises

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
function afficher(reservation)
{
	document.getElementById(reservation).style.visibility = "";
}
function cacher(reservation)
{
	document.getElementById(reservation).style.visibility = "hidden";
}

Des quotes ici car c'est une chaine que tu passes en parametre et un point-virgule à la fin, c'est plus propre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

onClick='afficher("reservation");'

Pour eviter de confondre parametre, chaine et variable, tu devrais plutot l'ecrire comme ça :
(c'est la même chose, y'a juste les noms de changé)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
function afficher(elemID)
{
	document.getElementById(elemID).style.visibility = "";
}
function cacher(elemID)
{
	document.getElementById(elemID).style.visibility = "hidden";
}

elemID pour "element ID" histoire d'être cohérent avec l'utilisation faite de ce parametre

merci!!!!!
ça a l'air si simple quand on a la solution!
deux heures que j'essaie (pas habitué du javascript... )
encore merci!