Hola:
Lo de poner "#" en un enlace es en realidad para enlazar al tope de la página, pero no creo que eso sea lo que pretendes... lo que hace un enlace es siempre enlazar, a no ser que se cancele enviando el valor false al evento asociado:
onclick="return false"...
Pienso que lo estabas haciendo bien, porque enlazabas a esa imagen, pero te falta el "efecto" que quieres, que puede hacerse de dos maneras: poner la imagen como fondo de esa capa, o insertar una etiqueta imagen; y de los 2, yo me decantaría por el primero porque evitarías el código para crear y borrar elementos, y además, sería más fácil su posicionamiento. Por ejemplo añadiendo en la declaración de estilos, los ídem background-position (por ejemplo con "center center"), y background-repeat con "no_repeat".
Luego simplemente te quedaría poner esa imagen como fondo de la capa:
Código:
onclick="document.getElementById('cp_contenedor_miniaturas').style.backgroundImage = 'url(" + this.href + ")'; return false"
Saludos