@MarcePuente No domino como tal JS, pero te puedo decir que ese código (el original) está lleno de errores... Por cierto, te recomiendo leer
este artículo. Se desaconseja usar manejadores de eventos
inline. Igual, uno es libre de seguir haciéndolo, como seguir usando <center></center> :)