Discussion :

[Nuks1304]

Bonjour tout le monde,

j'ai un petit soucis avec Firefoxe j'ai regardé sur le fofo mais pas trouvé :/

le code suivant fonctionne avec IE, Safarie, chrome mais pas avec FF

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
<script language="javascript">
 
function test(a) {
 
var x = (a.value || a.selectedIndex.value);
var Solde =   "<%=Session("SOLDE")%>";
var Style =   "width: 820px; left: 10px; margin-left: 10px; margin-top: 0px;";
var Style2 =  "width: 800px; left: 5px; margin-left: 5px; margin-top: 0px;";
var Img =     "Images/mini/solde.png"
var Img2 =    "Images/mini/solde2.png"
var Label =   document.getElementById("Label")
var Photo =   document.getElementById("Photo")
var Desc =    document.getElementById("Desc")
 
	switch (x) {
 
		case "Rose":
 
			if (Solde == 0)
			{		
		  		Label = "10"; 
				Desc.setAttribute("style", Style);
				Photo.setAttribute("src", Img);
		  		break;
			}
			else
			{
		  		Label = "20"; 
				Desc.setAttribute("style", Style2);
				Photo.setAttribute("src", Img2);
				break;
			}
 
	}
 
}

j'appel ma fonction depuis une liste déroulante

Code :

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

<select id="liste" onchange="test(this)" name="Couleur" style="width: 172px">

J'espère que vous saurez m'aiguillez car là je sais plus ou regarder

Cordialement
Emilio

[andry.aime]

Bonjour,

Code :

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

1
2
3
var Solde =   "<%=Session("SOLDE")%>";
...
if (Solde == 0)

Solde est de type String et tu la compares avec un type entier.

A+.

[SpaceFrog]

Code :

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

<script language="javascript">

Un script de 1932 ?

=>

Code :

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

<script type="text/javascript">

[Nuks1304]

Bonjour

Merci pour vos réponse.

andry.aime effectivement, j'ai corrigé.

SpaceFrog ahah oui c'est corrigé aussi

Mais malheureusement ça ne fonctionne toujours pas

Une autre idée ?

Cordialement
Emilio

[raton_laveur]

Sans vouloir t'offenser, tu nous donnes un bout de code (partiel), sans nous expliquer le problème, à part savoir qu'il ne marche pas sous FF. Perso, j'ai du mal à savoir de quel erreur tu parles.

Par curiosité, <%=Session("SOLDE")%> c'est du langage serveur ?

[Nuks1304]

Coucou raton_laveur

Oui tu as raison désolé

En fait ce petit bout de code me sert à changer une valeur par exemple:

Quand je clique sur ma liste déroulant et que je choisis une valeur ça me lance cette fonction qui me permet de changer une image, si je choisis la couleur Rose ça me charge mon image rose ainsi de suite.

Oui c’est de l’ASP.

EDIT: en fait le problème c'est que ce code n'est pas déclenché par FF.

Cordialement
Emilio

[andry.aime]

a.selectedIndex.value

Un selectedIndex est de type entier, il n'a pas d'attribut value.

[Nuks1304]

Ok merci, c'est corrigé.

C'est a devenir fou quand même

[SpaceFrog]

Code :

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

a.selectedIndex.value

=>

Code :

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

a.options[selectedIndex].value

[Nuks1304]

Corrigé mais fonctionne toujours pas sur FF :/

est ce que l'appel de la fonction peut poser problème ?

Code :

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

onchange="test(this);"

car j'ai lu qu'il y a des navigateurs qui posent problèmes avec la propriété this.

[andry.aime]

Code :

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

var x = a.options[a.selectedIndex].value;

Sinon, peut on voir le code HTML généré?

[andry.aime]

Et ajoutes le cas "default : break;" pour le switch pour éviter une boucle infinie.

[Nuks1304]

voilà le script généré, ça fonctionne toujours partout sauf sur FF

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
 
<script type="text/javascript">
 
function VerifCouleur() {
 
var ChoixCouleur = document.getElementById("ChoixListe");
 
var Choix  = (ChoixCouleur.value || ChoixCouleur.selectedIndex);
 
var solde1 = 0;
var solde2 = 116.1;
var solde3 = 0;
var solde4 = 0;
var solde5 = 0;
 
var prix1 =  129;
var prix2 =  129;
var prix3 =  129;
var prix4 =  0;
var prix5 =  0;
 
var Couleur1 = "Rose";
var Couleur2 = "Noir";
var Couleur3 = "Violet";
var Couleur4 = "";
var Couleur5 = "";
 
var style =  "width: 820px; left: 10px; margin-left: 10px; margin-top: 0px;";
var style2 = "width: 820px; left: 10px; margin-left: 10px; margin-top: 0px;";
var style3 = "background-image: url('Images/solde.png'); background-position: center; background-repeat: no-repeat; height: 120px";
var style4 = "height: 0px";
 
var imgProCoul1 = "Images/mini/1100236000000.png"
var imgProCoul2 = "Images/mini/1100237000000.png"
var imgProCoul3 = "Images/mini/1100251000000.png"
var imgProCoul4 = "Images/mini/.png"
var imgProCoul5 = "Images/mini/.png"
 
var Label1 =   document.getElementById("Label1")
var Label2 =   document.getElementById("Label2")
var Label3 =   document.getElementById("Label3")
var DescProd = document.getElementById("DescProd")
var imgSolde = document.getElementById("imgSolde")
var photo =    document.getElementById("photo")
 
	switch (Choix) {
 
		case Couleur1 :
 
			if (solde1 == 0)
			{		
		  		document.all.Label1.innerHTML="Prix "+"129"+".-"; 
		  		document.all.Label2.innerHTML=''; 
		  		document.all.Label3.innerHTML=''; 
				document.getElementById("DescProd").setAttribute("style", style2);
				document.getElementById("imgSolde").setAttribute("style", style4);
				document.getElementById("photo").setAttribute("src", imgProCoul1);
				break;
			}
			else
			{
		  		document.all.Label1.innerHTML=''; 
				document.all.Label2.innerHTML="Prix "+"129"+".-"; 
				document.all.Label3.innerHTML="Prix "+"116"+".-"; 
				document.getElementById("DescProd").setAttribute("style", style);
				document.getElementById("imgSolde").setAttribute("style", style3);
				document.getElementById("photo").setAttribute("src", imgProCoul1);
				break;
			}
 
 
		case Couleur2 :
 
			if (solde2 == 0)
			{		
		  		document.all.Label1.innerHTML="Prix "+"129"+".-"; 
		  		document.all.Label2.innerHTML=''; 
		  		document.all.Label3.innerHTML=''; 
				document.getElementById("DescProd").setAttribute("style", style2);
				document.getElementById("imgSolde").setAttribute("style", style4);
				document.getElementById("photo").setAttribute("src", imgProCoul2);
				break;
			}
			else
			{
		  		document.all.Label1.innerHTML=''; 
				document.all.Label2.innerHTML="Prix "+"129"+".-"; 
				document.all.Label3.innerHTML="Prix "+"116"+".-"; 
				document.getElementById("DescProd").setAttribute("style", style);
				document.getElementById("imgSolde").setAttribute("style", style3);
				document.getElementById("photo").setAttribute("src", imgProCoul2);
				break;
			}
 
 
		case Couleur3 :
 
			if (solde3 == 0)
			{		
		  		document.all.Label1.innerHTML="Prix "+"129"+".-"; 
		  		document.all.Label2.innerHTML=''; 
		  		document.all.Label3.innerHTML=''; 
				document.getElementById("DescProd").setAttribute("style", style2);
				document.getElementById("imgSolde").setAttribute("style", style4);
				document.getElementById("photo").setAttribute("src", imgProCoul3);
				break;
			}
			else
			{
		  		document.all.Label1.innerHTML=''; 
				document.all.Label2.innerHTML="Prix "+"129"+".-"; 
				document.all.Label3.innerHTML="Prix "+"0"+".-"; 
				document.getElementById("DescProd").setAttribute("style", style);
				document.getElementById("imgSolde").setAttribute("style", style3);
				document.getElementById("photo").setAttribute("src", imgProCoul3);
				break;
			}
 
 
		case Couleur4 :
 
			if (solde4 == 0)
			{		
		  		document.all.Label1.innerHTML="Prix "+"0"+".-"; 
		  		document.all.Label2.innerHTML=''; 
		  		document.all.Label3.innerHTML=''; 
				document.getElementById("DescProd").setAttribute("style", style2);
				document.getElementById("imgSolde").setAttribute("style", style4);
				document.getElementById("photo").setAttribute("src", imgProCoul4);
				break;
			}
			else
			{
		  		document.all.Label1.innerHTML=''; 
				document.all.Label2.innerHTML="Prix "+"0"+".-"; 
				document.all.Label3.innerHTML="Prix "+"0"+".-"; 
				document.getElementById("DescProd").setAttribute("style", style);
				document.getElementById("imgSolde").setAttribute("style", style3);
				document.getElementById("photo").setAttribute("src", imgProCoul4);
				break;
			}
 
 
		case Couleur5 :
 
			if (solde5 == 0)
			{		
		  		document.all.Label1.innerHTML="Prix "+"0"+".-"; 
		  		document.all.Label2.innerHTML=''; 
		  		document.all.Label3.innerHTML=''; 
				document.getElementById("DescProd").setAttribute("style", style2);
				document.getElementById("imgSolde").setAttribute("style", style4);
				document.getElementById("photo").setAttribute("src", imgProCoul5);
				break;
			}
			else
			{
		  		document.all.Label1.innerHTML=''; 
				document.all.Label2.innerHTML="Prix "+"0"+".-"; 
				document.all.Label3.innerHTML="Prix "+"0"+".-"; 
				document.getElementById("DescProd").setAttribute("style", style);
				document.getElementById("imgSolde").setAttribute("style", style3);
				document.getElementById("photo").setAttribute("src", imgProCoul5);
				break;
			}
	default : break;	
	}
 
}
</script>

[andry.aime]

var Choix = (ChoixCouleur.value || ChoixCouleur.selectedIndex);

Code :

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

var Choix  = (ChoixCouleur.options[ChoixCouleur.selectedIndex].value);

document.all

ça ne fonctionne que sous IE et c'est même déjà trop vieux.

[NoSmoking]

Bonjour,
à quoi sert-il de récupérer les éléments, vu dans le code

Code :

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

1
2
3
4
5
6
var Label1 =   document.getElementById("Label1")
var Label2 =   document.getElementById("Label2")
var Label3 =   document.getElementById("Label3")
var DescProd = document.getElementById("DescProd")
var imgSolde = document.getElementById("imgSolde")
var photo =    document.getElementById("photo")

pour ne pas les utiliser?

[Nuks1304]

Bonjour NoSmoking,

je suis entrain de modifier le code pour le faire plus "propre"

Bravo andry.aime depuis que tu m'a dit de modifier "document.all" ça fonctionne sur FF MAIS ^^ mes labels ne fonctionne plus nul part. comment faire pour qu'il prennent la valeur demandée ?

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
<script type="text/javascript">
 
function VerifCouleur() {
 
var ChoixCouleur = document.getElementById("ChoixListe");
 
var Choix  = (ChoixCouleur.options[ChoixCouleur.selectedIndex].value);
 
var solde1 = 0;
var solde2 = 116.1;
var solde3 = 0;
var solde4 = 0;
var solde5 = 0;
 
var prix1 =  129;
var prix2 =  129;
var prix3 =  129;
var prix4 =  0;
var prix5 =  0;
 
var Couleur1 = "Rose";
var Couleur2 = "Noir";
var Couleur3 = "Violet";
var Couleur4 = "";
var Couleur5 = "";
 
var style =  "width: 820px; left: 10px; margin-left: 10px; margin-top: 0px;";
var style2 = "width: 820px; left: 10px; margin-left: 10px; margin-top: 0px;";
var style3 = "background-image: url('Images/solde.png'); background-position: center; background-repeat: no-repeat; height: 120px";
var style4 = "height: 0px";
 
var imgProCoul1 = "Images/mini/1100236000000.png"
var imgProCoul2 = "Images/mini/1100237000000.png"
var imgProCoul3 = "Images/mini/1100251000000.png"
var imgProCoul4 = "Images/mini/.png"
var imgProCoul5 = "Images/mini/.png"
 
var Label1 =   document.getElementById("Label1")
var Label2 =   document.getElementById("Label2")
var Label3 =   document.getElementById("Label3")
 
var DescProd = document.getElementById("DescProd")
var imgSolde = document.getElementById("imgSolde")
var photo =    document.getElementById("photo")
 
	switch (Choix) {
 
		case Couleur1 :
 
			if (solde1 == 0)
			{		
		  		Label1.value ="Prix 129.-"; 
		  		Label2.value =''; 
		  		Label3.value =''; 
				DescProd.setAttribute("style", style2);
				imgSolde.setAttribute("style", style4);
				photo.setAttribute("src", imgProCoul1);
				break;
			}
			else
			{
		  		Label1.value =''; 
				Label2.value ="Prix 129.-"; 
				Label3.value ="Prix 116.-"; 
				DescProd.setAttribute("style", style);
				imgSolde.setAttribute("style", style3);
				photo.setAttribute("src", imgProCoul1);
				break;
			}
 
 
		case Couleur2 :
 
			if (solde2 == 0)
			{		
		  		Label1.text ="Prix 129.-"; 
		  		Label2.text =''; 
		  		Label3.text =''; 
				DescProd.setAttribute("style", style2);
				imgSolde.setAttribute("style", style4);
				photo.setAttribute("src", imgProCoul2);
				break;
			}
			else
			{
		  		Label1.text =''; 
				Label2.text ="Prix "+"129"+".-"; 
				Label3.text ="Prix "+"116"+".-"; 
				DescProd.setAttribute("style", style);
				imgSolde.setAttribute("style", style3);
				photo.setAttribute("src", imgProCoul2);
				break;
			}
 
 
		case Couleur3 :
 
			if (solde3 == 0)
			{		
		  		Label1.value ="Prix "+"129"+".-"; 
		  		Label2.value =''; 
		  		Label3.value =''; 
				DescProd.setAttribute("style", style2);
				imgSolde.setAttribute("style", style4);
				photo.setAttribute("src", imgProCoul3);
				break;
			}
			else
			{
		  		Label1.value =''; 
				Label2.value ="Prix "+"129"+".-"; 
				Label3.value ="Prix "+"0"+".-"; 
				DescProd.setAttribute("style", style);
				imgSolde.setAttribute("style", style3);
				photo.setAttribute("src", imgProCoul3);
				break;
			}
 
 
		case Couleur4 :
 
			if (solde4 == 0)
			{		
		  		Label1.value ="Prix "+"0"+".-"; 
		  		Label2.value =''; 
		  		Label3.value =''; 
				DescProd.setAttribute("style", style2);
				imgSolde.setAttribute("style", style4);
				photo.setAttribute("src", imgProCoul4);
				break;
			}
			else
			{
		  		Label1.value =''; 
				Label2.value ="Prix "+"0"+".-"; 
				Label3.value ="Prix "+"0"+".-"; 
				DescProd.setAttribute("style", style);
				imgSolde.setAttribute("style", style3);
				photo.setAttribute("src", imgProCoul4);
				break;
			}
 
 
		case Couleur5 :
 
			if (solde5 == 0)
			{		
		  		Label1.value ="Prix "+"0"+".-"; 
		  		Label2.value =''; 
		  		Label3.value =''; 
				DescProd.setAttribute("style", style2);
				imgSolde.setAttribute("style", style4);
				photo.setAttribute("src", imgProCoul5);
				break;
			}
			else
			{
		  		Label1.value =''; 
				Label2.value ="Prix "+"0"+".-"; 
				Label3.value ="Prix "+"0"+".-"; 
				DescProd.setAttribute("style", style);
				imgSolde.setAttribute("style", style3);
				photo.setAttribute("src", imgProCoul5);
				break;
			}
	default : break;	
	}
 
}
</script>

PS: il y a dans le code Label1.value, et Label1.text car je test pour trouver la propriété qui accepte la valeur

[NoSmoking]

Pour les label il te faut modifier, comme dans ton code initial, le innerHTML.

[Nuks1304]

Bon bha je sais pas ce que j'aurai fait sans vous !

Tout fonctionnent impec sur tout les navigateurs

Vous aurez bien payé une bière là

En tout cas merci pour votre aide et votre patience, Merci beaucoup.

et ce document.all vais pas l'oublier de si tôt !

Cordialement
Emilio