2016-05-27 10 views
1

Ich möchte eine img wo die onclick versteckt eine andere img ID und zeigt eine andere. Dies ist der Code, den ich gemacht:Hinzufügen onclick Ereignis zu 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>

Wenn ich mouseover verwenden, die Funktion funktioniert, aber mit onclick nicht. Was mache ich falsch?

+1

Es funktioniert hier: https://jsfiddle.net/Lskfyemy/1/, aber wir haben eine Javascript Fehler wenn '# textpolu' nicht existiert – Pierre

+0

was ist" textpolu "? – santon

+0

Warum nicht jquery toggle() verwenden? –

Antwort

1

Eine weitere Option ist Jquery verwenden toggle() siehe Beispiel 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

Funktioniert und das ist sauberer als das, was ich hatte, danke –

+0

@AndreFIlipe Ich dachte, du hast gesagt, du bist nicht soll jQuery verwenden? – Barmar

+0

@Barmar Solange ich eine gute Begründung gebe, ist es in Ordnung, egal, ich habe den alten Code kommentiert und korrigiert, wo ich falsch gemacht habe –