2016-05-27 1 views
1

Je souhaite créer un img où le onclick cache un autre identifiant et affiche un autre. Voici le code que j'ai fait:Ajout de l'événement onclick à img id

document.getElementById("poluzoom").addEventListener("click", changeimg); 
 

 
function changeimg() { 
 
    document.getElementById("poluzoom").style.visibility = "visible"; 
 
    document.getElementById("readypol").style.visibility = "hidden"; 
 
    document.getElementById("textpolu").style.visibility = "hidden"; 
 
}
<div id="fabricas"> 
 
    <img id="poluzoom" src="fabrica/fabricas.gif" alt="Mundo Zoom"> 
 
    <img id="readypol" src="poluicao/offbuton.png" alt="but1"> 
 
</div>

Si je mouseover, la fonction fonctionne, mais avec onclick il ne fonctionne pas. Qu'est-ce que je fais mal?

+1

Il travaille ici: https://jsfiddle.net/Lskfyemy/1/, mais nous avons un javascript erreur si '# textpolu' n'existe pas – Pierre

+0

Qu'est-ce que" textpolu "? – santon

+0

Pourquoi ne pas utiliser jquery toggle()? –

Répondre

1

Une autre option est d'utiliser Jquery toggle() voir exemple http://jsfiddle.net/M9QBb/434/

<div id="fabricas"> 
    <img id="poluzoom" src="http://lorempixel.com/400/200/" alt="Mundo Zoom"> 
    <img id="readypol" src="http://lorempixel.com/400/200/sports/" alt="but1" style="display: none"> 
</div> 

JQuery

$("#fabricas").click(function() { 
    $(this).find('img').toggle(); 
}); 
+0

Fonctionne et c'est plus propre que ce que j'avais, merci –

+0

@AndreFIlipe Je pensais que tu disais que tu n'es pas censé utiliser jQuery? – Barmar

+0

@Barmar Tant que je donne une bonne justification c'est bien, de toute façon, j'ai gardé l'ancien code commenté et corrigé où je me trompais –