Eventos en Javascript

Cruzado51 · #1 (**permalink**) 22/12/2011, 04:00

Buenas, tengo un problema con este script en la mayoría de los navegadores
(en Chrome me funciona)
El problema esta en que no se me inicia el evento onClick (ningún evento me funciona)

window.onload=iniciar;

function iniciar()
{
document.write("<img src='imagenes/1.jpg' onClick='iniciar2();' alt='Imagen' />");
}

function iniciar2()
{
alert("Hola");
}

Gracias de Antemano
Saludos

IsaBelM · #2 (**permalink**) 22/12/2011, 06:35

el problema es que document.write sobre escribe todo el contenido. por lo tanto has de usar document.getElementById('foto').innerHTML

Cruzado51 · #3 (**permalink**) 22/12/2011, 12:26

Gracias por contestar IsaBelM

La cuestión es que, quiero crear casi todo el html desde javascript y por eso añado esa linea

Y al no tener en la página principal (.html) la linea <img src="imagenes/1.jpg" id="imagen" /> , ese código no me va a funcionar

Saludos

Dnielf · #4 (**permalink**) 22/12/2011, 12:51

No te va a funcionar el evento onClick porque solo estás escribiendo el elemento mas no lo agregas al DOM, en realidad tienes que crearlo así :

Código:

var img = document.createElement("img");
img.src = "imagenes/1.jpg";
img.onclick = inicar2();
img.setAttribute("alt","Imagen");
document.body.appendChild(img);

ryugen · #5 (**permalink**) 22/12/2011, 13:27

perdon por el offtopic, pero no creo que "document.write" se la forma de crear un documento en javascript

marlanga · #6 (**permalink**) 22/12/2011, 13:43

Como ya te han dicho, si haces document.write, estás eliminando todo el código de la página, <HTML><HEAD><SCRIPT><BODY>, etc, escribiendo sólo el <IMG...>, asi que cuando se dispare su onclick, obviamente no encontrará nada.

La solución es usar, por ejemplo:

Código HTML:

<html>
	<head>
		<title>Prueba</title>

		<script>
			
			function inicio(){
				var img=new Image();
				img.width=100;
				img.height=100;
				img.onclick=inicio2;
				document.body.appendChild(img);
			}
			function inicio2(){
				alert("funciona");
			}
		</script>
	</head>

	<body onload="inicio();">
		
	</body>
</html>

Cruzado51 · #7 (**permalink**) 22/12/2011, 16:10

Dnielf y marlanga

Muchísimas Gracias