SocialLinksProfile with CSS grid and flex

Community feedback

• 

  Adriano•38,250

  @AdrianoEscarabotePosted about 1 month ago

  Hi brian, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

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

  I hope you find it useful. 👍

  0

  ---

  

  P

  Brian•110

  @coding-brianPosted about 1 month ago

  @AdrianoEscarabote Hi,Adriano, Thanks for your insight and the suggestion is constructive. I'm appreciated that you post so clear feedback for me. Really useful and I will improve my code. Thanks agian

  1

  

  Adriano•38,250

  @AdrianoEscarabotePosted about 1 month ago

  @coding-brian No problem !! Keep coding

  0
• 

  iliassabbad•40

  @iliassabbadPosted about 1 month ago

  look nice

  0

  ---

  

  P

  Brian•110

  @coding-brianPosted about 1 month ago

  @iliassabbad Thanks.

  0