davidw93/mozillians — comment on commit 79aa24f52edc52d598c3361bd1584d51cc3421d0
File:  line 0
By tallowen:

This looks great and it seems to do the job. However, there are some things about this code that is making making me uneasy. The background images are weirdly defined as a background to a div in a [different spot](https://github.com/mozilla/mozillians/blob/79aa24f52edc52d598c3361bd1584d51cc3421d0/media/css/main.css#L397). Though this is not your fault, putting a link in a different spot only makes it more confusing. Lets use an image tag for the images instead of this .imglink class.

Btw don't worry about rebasing your changes till after you're done. We can try working through that together when we get to that point.