David S• 150
@davsanter1
Posted
Hi! Your code looks good! The only thing that came to mi mind was to adjust the semantic HTLM replacing the <div> for the container, like <main>.
Also the <ul> and <li> for the social links with <nav> or a <div>, and use <a> tags instead of buttons for links. Example:
<nav> <a href="#">GitHub</a> <a href="#">Frontend Mentor</a> <a href="#">LinkedIn</a> <a href="#">Twitter</a> <a href="#">Instagram</a> </nav>
Hope it helps! :)
0
@balajisudarshan
Posted
@davsanter1 thanks for the tip
1
