Discussion :

[spoker04]

Bonjour,

J'ai réaliser le code suivant afin d'afficher/masquer une vidéo à chaque fois que l'on clique sur le lien associé, mais cela ne fonctionne pas du tout :

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
 
<script type="text/javascript">
 
    var voir_video = 0;
 
	function AfficherVideo()
	{
		if (voir_video==0) {
	  		document.getElementById('video_youtube').style.display = "block";
	  		voir_video=1; }
	  	if (voir_video==1) {
	  		document.getElementById('video_youtube').style.display = "none";
	  		voir_video=0; }
	}
</script>
-
-
-
<a href="#" onClick="AfficherVideo()">Video Youtube</a>
<div id="video_youtube" style="display:none;"><?php echo  $_product->getData('video_youtube'); ?></div>

Quelqu'un peut-il m'expliquer où se trouve mon erreur ? Peut être existe-il une solution plus simple...

Par avance, merci de votre aide

[Bovino]

Au début de ton script, voir_video est égal à 0, donc au premier clic, tu passes dans la condition

Code :

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

if(voir_video==0)

et dans celle-ci tu affectes la valeur 1 à voir_video, donc tu vas aussi passer dans la condition suivante

Code :

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

if(voir_video==1)

[spoker04]

Ah ok je comprend, j'aurais du faire une boucle if/else !

J'ai fianlement trouver ce code qui fonctionne très bien :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function AfficherVideo(thingId)
{
	var targetElement;
	targetElement = document.getElementById(thingId) ;
	if (targetElement.style.display == "none")
	{
		targetElement.style.display = "" ;
	} else {
		targetElement.style.display = "none" ;
	}
}

Merci pour ta réponse.

[vermine]

Bonjour,

D'abord if/else n'est pas une boucle.
Ensuite, ceci n'est pas certifié même si il fonctionnera sur bien des navigateurs:

Code :

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

1
2
 
targetElement.style.display = "" ;

Vous êtes censé préciser une valeur comme inline ou bien block (les plus courantes, mais il y en a bien d'autres.