html css

Community feedback

• 

  Dylan•290

  @dquinn089

  Posted 11 months ago

  Nice work! The only thing is that this is suppose to be a profile to consolidate social media links, but your code doesn't include an a tag to incorporate these links. You should us ul and li instead of the div elements, as well as including an a tag inside the li elements.

  Instead of:

  <div class="social_media">
          <div class="social_media_account">GitHub</div>
          <div class="social_media_account">Frontend Mentor</div>
          <div class="social_media_account">LinkedIn</div>
          <div class="social_media_account">Twitter</div>
          <div class="social_media_account">Instagram</div>
  </div>
  

  do:

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

  Hope this helps, happy coding!

  01