[SOLUCIONADO] Adaptar a navegadores modernos una página hecha hace años para IE7

[WiperWoper]

Buenos días,

Intentaré ser breve. Hace 8 años hice un curso de desarrollo web que incluía javascript y por aquel entonces hice una especie de juego de carreras de caballos que, de aquella, funcionaba en Internet Explorer 7. Hoy en día sin embargo no funciona en ningún navegador moderno.

No he vuelto a tocar javascript por lo que lo tengo bastante olvidado, pero con Firebug he podido ver cuál es el fallo (al menos el primero de ellos, lo mismo aparecen más al solucionar este) que me está dando.

En este trozo de código me dice que p is null:

Código Javascript:

p=findDOM("tiempouno",0);
...
p.value=total1;
 

La función findDOM la usaba para determinar la forma de recuperar un elemento según el navegador en el que se ejecutase, y es esta:

[URL="http://pastebin.com/fVauWhPA"]http://pastebin.com/fVauWhPA[/URL]

Y "tiempouno" es un textbox de un formulario declarado de esta forma:

Código HTML:

<td><input type="text" name="tiempouno" disabled>ms</td> 

¿Alguna idea de lo que puede estar pasando? Si se necesita el código entero lo subo sin problemas, pero es algo largo así que he preferido empezar con esto a ver si es suficiente y os ahorro buscar entre 400 lineas.

Gracias de antemano.

[PHPeros]

Yo dejaría de dar soporte a navegadores tan antiguos, y empezaría a utilizar el DOM de la W3C.

[WiperWoper]

Iniciado por PHPeros

Yo dejaría de dar soporte a navegadores tan antiguos, y empezaría a utilizar el [URL="https://developer.mozilla.org/en-US/docs/Using_Web_Standards_in_your_Web_Pages/Using_the_W3C_DOM"][B]DOM de la W3C[/B][/URL].

No es mi intención darles soporte ni mucho menos a los navegadores antiguos, son simplemente los que existían cuando hice esa página.

Probaré de todas formas a sustituir las llamadas a la función findDOM por getElementById a ver si con eso consigo que funcione en navegadores actuales.

[WiperWoper]

Lo de cambiar el findDOM por getElementById no ha arreglado gran cosa. Ahora al pulsar el botón para que comience la carrera, en vez de dar el error que daba antes de p is null simplemente no da ningún error por consola ni por ningún lado y lo único que ocurre es que desaparece el botón.

Edit: vale lo de hacer desaparecer el botón por lo visto es parte del código, eso no es un error. Pero vamos que los caballos siguen sin moverse, le seguiré echando un ojo por mi cuenta porque es demasiado código como para pediros que lo reviséis aquí.

[PHPeros]

¿Y ahora qué error te da? ¿Caballos are null?

[WiperWoper]

jaja no, no da ningún error, simplemente no se mueven.

Peeeero ya encontré el problema a base de debugear con Firebug. El problema era que para mover los caballos utilizaba esto básicamente:

Código Javascript:

a=document.getElementById("caballo1").style;
a.left=posx+aleatorio;
 

posx+aleatorio es un número de pixels, pero simplemente eso, el número. Por lo visto eso funcionaba hasta IE7, pero ahora para que funcione hay que poner "px", así que ha quedado así:

Código Javascript:

a=document.getElementById("caballo1").style;
a.left=(posx+aleatorio)+"px";
 

No lo he mirado demasiado a fondo aún pero parece que con eso ya está arreglado y funciona en Firefox 35 al menos. Marco el tema como solucionado :)

Gracias de todas formas porque me da que la función findDOM está desfasada y no habría funcionado sin cambiarlo por getElementById.