social links profile

Please log in to post a comment
Log in with GitHubCommunity feedback
- P@AnjelToppo
Well, from my side I'm unable to see your live site. Seems like it's broken.
Some points you might consider to enhance the semantics-
-
Inside body always provide at least one <main> tag.
-
You can wrap this code inside <nav> tag and also introduce <a> tag.
BEFORE
<ul class="social__links"> <li>GitHub</li> <li>Frontend Mentor</li> <li>LinkedIn</li> <li>Twitter</li> <li>Instagram</li> </ul>
AFTER
<nav class="width-100per"> <ul class="links flex fd-column"> <li><a href="#" target="_blank" rel="noopener noreferrer">Github</a></li> <li><a href="#" target="_blank" rel="noopener noreferrer">Frontend Mentor</a></li> <li><a href="#" target="_blank" rel="noopener noreferrer">LinkedIn</a></li> <li><a href="#" target="_blank" rel="noopener noreferrer">Twitter</a></li> <li><a href="#" target="_blank" rel="noopener noreferrer">Instagram</a></li> </ul> </nav>
-
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord