Hola:
Lo que no estoy seguro de que esté bien es "document.onclick"... creo que debería ser window.onclick o document.body.onclick...
Para asegurarte te recomiendo probar antes con alertas el funcionamiento del evento (según parece lo demás va bien...)
Saludos
