Responsive social profile using flex properties

Please log in to post a comment
Log in with GitHubCommunity feedback
- @MahmoodHashem
Hey there!
Congrats on finishing the challenge! ✅
Your solution looks awesome. Here are a few suggestions to further improve your code:
- Use Semantic HTML : Instead of using buttons for navigation, consider using
<a>
tags within a<ul>
and<li>
structure and<h1>
...<h6>
for headings . This provides better semantic meaning and accessibility for your content.
<ul class="list-container"> <li><a href="#">Github</a></li> <li><a href="#">Frontend Mentor</a></li> <li><a href="#">LinkedIn</a></li> <!-- Add more list items as needed --> </ul>
Explanation: Using
<ul>
and<li>
elements creates a semantic list structure, which is more appropriate for navigation links. This improves the accessibility and understandability of your HTML markup.Keep up hard work!
Marked as helpful - Use Semantic HTML : Instead of using buttons for navigation, consider using
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