updated Social Page Challenge / CSS flexbox

Community feedback

• 

  Adriano•38,270

  @AdrianoEscarabotePosted about 1 month ago

  Hello halovii, how are you? I was really pleased with your project, but I’d like to offer some advice that might help:

  To enhance the semantics of your code, consider using a <ul> (unordered list) for the collection of links, as it represents a list of related items. Here's an example:

  <ul>
      <li><a href="#">GitHub</a></li>
      <li><a href="#">Frontend Mentor</a></li>
      <li><a href="#">LinkedIn</a></li>
      <li><a href="#">Twitter</a></li>
      <li><a href="#">Instagram</a></li>
  </ul>
  

  Using a <ul> provides clear structure and context, signaling to both browsers and assistive technologies that these links are part of a cohesive group, improving both accessibility and readability.

  The rest is spot on.

  Hope it’s helpful to you. 👍

  Marked as helpful

  1

  ---

  

  halovii•70

  @haloviiPosted about 1 month ago

  Thank you! I'll try this in future projects, I really appreciate your help😊@AdrianoEscarabote

  0

  

  Adriano•38,270

  @AdrianoEscarabotePosted about 1 month ago

  @halovii No problem ! keep codingg

  0
• 

  M Asim Saeed•270

  @asimsaeed353Posted about 1 month ago

  Impressive effort and yeah GitHub is confusing sometimes haha. I have a suggestion, add styles to your <a> for hover and focused state. That'll improve the design for sure. Keep up the good work!

  1