@mattstuddert
Posted
Awesome work on this challenge, it looks great! 👍
Your HTML is really solid and your Sass looks good. One thing I would say is that with Sass you're able to nest media queries instead of adding them all at the bottom of the file. This makes managing the different layouts much easier, as all the styles for an element are in one place.
Also, I'd recommend using min-width
media queries instead of max-width
. They have a number of benefits, including optimising the loaded CSS for mobile users.
I hope those pointers help!