Responsive social profile using flex properties

Community feedback

• 

  Mahmood•1,450

  @MahmoodHashemPosted 4 months ago

  Hey there!

  Congrats on finishing the challenge! ✅

  Your solution looks awesome. Here are a few suggestions to further improve your code:

  1. 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

  1