Hola:
Me parece mala práctica enlazar a "#" (o sea tope de la página... enlace interno) para lanzar un script... son enlaces innecesarios y no accesibles, ya que realmente deberías enlazar a esa imagen...
Lo del tope de la páginaa, es por lo que indiqué antes, href="#" equivale a enlazar internamente al tope de la página, y para evitarlo basta con cancelar ese enlace (onclick="return false")...
Si te interesa hacer el efecto lightbox, te muestro un apunte donde lo explico:
Lightbox sin librerías.
Saludos
