Dificultades con Mocilla

cgm-util · #1 (**permalink**) 20/12/2010, 22:26

Hola amigos:

Tengo un script en js que a continuación os presento el cual me funciona perfectamente en IE pero en Mocilla no.
Se que es algo antiguo y que tengo que emplear getElementById() pero no se como hacerlo. ¿me podeis ayudar?

<script language="JavaScript" type="text/javascript">
//PANTALLA DE DESPLAZAMIENTO PUBLICITARIO

var ns4 = (document.layers)? true:false
var ie4 = (document.all)? true:false
var objetoBandera
var retardo = 30
var pixels = 30

function Inicio()
{
if (ns4)
objetoBandera = document.divBandera
else
if (ie4)
objetoBandera = divBandera.style
}

function Animar()
{
if (parseInt(objetoBandera.top) < 1)
{
objetoBandera.top = parseInt(objetoBandera.top) + pixels
setTimeout("Animar()",retardo)
}
}

function Iniciar()
{
if (parseInt(objetoBandera.top) > -260){
objetoBandera.top = parseInt(objetoBandera.top) - pixels
setTimeout("Iniciar()",retardo)}
}

function Retardo (valor)
{
retardo = valor
}

function Pixels (valor)
{
pixels = valor
}

</script>
</head>
<body onLoad="presImagen()">

<script>setTimeout("Inicio()",1500)</script>
<script>setTimeout("Animar()",3000)</script>
<script>setTimeout("Iniciar()",13000)</script>

<script>
var direcciones = new Array('<div ID="divBandera" STYLE="position:absolute; left:5px; top:-260px; width:250;height:291;z-index:3"><embed src="flash/nuevopopup.swf" width="250" height="250"></embed></div>')
aleat = Math.random() * direcciones.length
aleat = Math.floor(aleat)
document.write(direcciones[aleat]);
</script>
</body>
</html>

pinchoso · #2 (**permalink**) 21/12/2010, 10:24

Te paso unas modificaciones de las funciones Js con las que en principio deberia funcionarte en Mz, fijate que he modificado alguna variable.

Y sobretodo, acostumbrate a usar getElementById... yo de hecho te recomendaria que a la larga vayas mirandote jQuery.

Código:

var objetoBandera;
var retardo = 3000;
var pixels = 30;

function Inicio(){
	objetoBandera = document.getElementById('divBandera');
}

function Animar(){
	if (parseInt(objetoBandera.style.top) < 1){
		objetoBandera.style.top = parseInt(objetoBandera.style.top) + pixels +"px";
		setTimeout("Animar()",retardo)
	}
}

function Iniciar() {
	if (parseInt(objetoBandera.style.top) > -260){
		objetoBandera.style.top = parseInt(objetoBandera.style.top) - pixels + "px";
		setTimeout("Iniciar()",retardo)
	}
}

cgm-util · #3 (**permalink**) 21/12/2010, 23:45

Eres extraordinario, he realizado tus modificaciones y funciona perfectamente. Muchisimas gracias.

Seguiré tus recomendaciones que me parecen muy oportunas.

Saludos.