@nzewi
Posted
Congrats on completing your first challenge JOAOVITORWITT
You have a great solution there
Here are my suggestions:
1.Use relative units like rem
and em
instead of px
for spacing and font-sizes, this would make your site more responsive. Learn about them and use them in your next project.
2.Ensure that all content on your page is contained within a landmark region, e.g Wrapping your container-inner <div>
inside a <main>
tag. This makes your HTML more semantic and accessible. You can use other semantic tags like button
instead of a div
for buttons. Also, make sure you have only 1 h1
on your page
I hope this helps
Marked as helpful
@joaovitorwitt
Posted
@nzewi thank you for the suggestions, I will definitely learn more about relative units.