Fraser Watt• 1,790
@fraserwat
Posted
Hey! This is looking great!!
Few things I'd change:
- Semantic HTML: your <div class="container"> acts more like a <main>, and I would have the <header> and <main> sections you currently have both as <section>. Also id's tend to be used for javascript functionality, I would use classes for the labelling.
- Gets a bit stretched out at larger screen widths. I would add
max-width: 1100px; margin: auto;
to the .container.
Keep up the good work!! Fraser
Marked as helpful
1