@janegca
Posted
Hi Michael, this one did have quite a lot to tackle, overall your page looks pretty good and responsive. Two small nits, the hover for the menu items is only a 1/2 line on the menu items and also, in the original design, the titles for the images is located at the bottom rather than the top of the image.
I don't regularly use SASS features but noticed the there is a lot of repetitive code in your CSS for loading the images, would a mixin work there? so you would only need to put in the url?
Nice job overall. Happy coding.
@mikeronewman
Posted
@janegca Thanks for the response! I noticed the image titles as soon as I submitted my solution. I didnt notice the hover underline width though, good eye. The repetitive background image code was extremely frustrating, so I think a mixin would be a great idea for that. Thanks for the input, cheers!