Problema básico sobre onclick event

[geclos]

Hola soy nuevo en esto de javascript y tengo unos problemas creando un menu desplegable.

La cuestion es que al apretar sobre el boton especifico para que se desplegue el menu la primera vez todo funciona muy bien, pero la segunda vez que apreto sobre el mismo boton se me carga el link del boton en vez del script.

Les pego el HTML y el script en concreto:

Código:

 <nav>
                    <a href="" id="popupmenulink">Nuestro trabajo</a>
		    		<a href="localhost">Sobre Nosotros</a>
		    		<a href="localhost">Blog</a></li>
		    		<a href="localhost">Contacto</a>
            </nav>
        </header>

        	<ul id="popupmenu">
        		<li>Proyectos</li>
        		<li>Servicios</li>
        	</ul>

Código:

window.onload = initAll;


function initAll() {

	var thismenu = document.getElementById('popupmenulink');
	thismenu.onclick = toggleMenu;
}

	function toggleMenu() {

	var thisMenu = document.getElementById('popupmenu').style;
	if (thisMenu.display == "block") {
		thismenu.display = "none";
	}
	else {
		thisMenu.display ="block";
	}
	return false;
}

gracias por la ayuda.

[laratik]

Es un tema bastante simple, fíjate en la siguiente linea de código:

Iniciado por geclos

Código:

if (thisMenu.display == "block") {
		thismenu.display = "none";
	}
	else {
		thisMenu.display ="block";
	}
}

Ahora recuerda que Javascript es case sensitive y thisMenu no es lo mismo que thismenu, así que thismenu no ha sido declarado, error que el interprete encuentra y detiene el script. Por lo tanto el return false nunca se ejecuta y pasa a la pagina.

Código Javascript:

if (thisMenu.display == "block") {
        thisMenu.display = "none";
    }
    else {
        thisMenu.display ="block";
    }
}
 

Sencillo cierto ... espero haberte sido de ayuda.

[geclos]

Tema solucionado pues.

Muchas gracias!