P
Matt Studdert• 13,611
@mattstuddert
Posted
Great work completing your 2nd challenge! 🙌
Here are a couple of pointers after taking a look at your code:
- You don't have a
h1
element on the page at the moment. Theh1
element is critical in marking up the most important heading on the page, so I would always recommend including one. - A lot of your images have both a
height
andwidth
declared on them. This can cause images to stretch or look squashed. Out of the two, I'd recommend setting thewidth
and then letting theheight
take care of itself. - Nice work going with
min-width
media queries! Working mobile-first is a great approach. You have got 4 separate media queries at1440px
at the moment. To make them more manageable, I'd recommend including all of those styles in a single1440px
media query at the bottom of your CSS file.
I hope that helps. Keep up the great work! 👍
0