Es muy buena practica usar sprites para las imágenes, a lo que se refiere Caricatos con una mala (yo diría pésima y horrorosa) practica, es el hecho de utilizar javascript para los links cuando no es necesario,
esto esta mal hecho:
Código HTML:
Ver original<li class="facebook" onClick="window.location.href('https://www.facebook.com/aimara.teatro')"><a href="https://www.facebook.com/aimara.teatro" title="facebook" target="_blank"></a></li>
debe ser
Código HTML:
Ver original<li class="facebook"><a href="https://www.facebook.com/aimara.teatro" title="facebook" target="_blank">Facebook
</a></li>
el evento javascript no viene al caso, es innecesario y un enlace no debe estar vació, si no quieres que se vea el texto utiliza algo como
Código CSS:
Ver original.facebook{display:block;width:30px;height:30px;overflow:hidden;text-indent:-100px;background:url(...);}
text-indent lo que hace es mover el texto 100px hacia la izquierda y al ser un elemento con ancho de solo 30px el texto queda fuera, ya solo se le aplica overflow:hidden; para que no muestre lo que esta fuera de los limites.