Fernando Batista• 270
@FernJBatista
Posted
Hi there!
I think you did great for this project!
The only suggestions I'd have for you is using semantic HTML. For example, you used a div and gave it the class "main-container", but you could just use the <main> tags and that also helps the browser understand where the main information is.
The only other thing I noticed is that you didn't use heading hierarchy, so you used h4 and h5 without using any of the higher hierarchy headings. It's good practice to have an h1 always and then follow the chain of headings. This was weird for me at the beginning, but it's more about the HTML structure than the styling.
Other than that, I think you did really good on this project!
Keep it up, happy coding!
Marked as helpful
2
