Skip to content
Submitted about 1 month ago

Social_link_profile_jessica

John Phillips•50
@John1999-debug
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social links profile
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I think I nailed this one... compared to the prior one

Community feedback

  • Adriano•38,390
    @AdrianoEscarabotePosted about 1 month ago

    Hi John Phillips, how’s everything? I think your project turned out great! However, I have some feedback that I think might be useful:

    A tip to improve the semantics of the code would be to use ul, since that is a list of links:

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

    The rest is amazing.

    I hope this is helpful. 👍

    1
    John Phillips•50
    @John1999-debugPosted about 1 month ago

    Hey @AdrianoEscarabote, this does look a lot better. Thanks for your feedback mate :)

    1

Please log in to post a comment

Log in with GitHub

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