Alex• 3,130
@Alex-Archer-I
Posted
Hi!
Great work, congrats! It's very neat and close to design =)
I can give you a couple suggestions about semantic. It's a list of links, so it's better to use ul tag here and put a inside li. Like this:
<ul>
<li><a href="#">It's a link here!</a></li>
...and more links here
</ul>
Than you doesn't need a button inside a - it's a tags with very different purposes =)
Also it is recommended not to skip headers hierarchy - I mean after the h1 should follow h2 etc.
Oh, and you can get rid of margin: auto for the main (good that you are use main tag, btw) since you already centered it with body tag.
Hope that helps. Keep doing great stuff =)
Marked as helpful
0
