Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social links profile solution

P

@josifermaodev

Desktop design screenshot for the Social links profile coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


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

I'm proud to have managed to come up with such a similar replica with so little experience!

What challenges did you encounter, and how did you overcome them?

My biggest challenge was knowing how to identify which tags to use and how to set up the link structure in HTML. I chose to use a list, it worked even though I didn't know if it was correct to do it that way.

What specific areas of your project would you like help with?

I would like help with the a tags, I would like an opinion on whether this is correct or if there is a simpler way to do it. thanks!

Community feedback

P

@Carson-Haskell

Posted

Amazing job, Josiane! That looks amazing. I have a few comments (since I am encouraged to give feedback :) )

  1. To better match the design, your profile picture should be a bit larger. I think it's supposed to be 58px x 58px or something like that. When doing the side-by-side, you can tell it's definitely smaller!
  2. On the hover state of your links, the text color is supposed to change to black. It's really hard to read the white text against the green background!
  3. I think your solution of using an unordered list totally works, but it would be easier and more semantic if you used a <nav> tag and then nested <a> tags for your links. For example:
<nav>
    <a href="/linktoyoursite.com" alt="link to ____ site">Website</a>
    <a href="/anotherlinktoanothersite.com" alt="same thing">Another Website</a>
</nav>

It is common convention to use <a> (link) tags for anything that takes you somewhere else (i.e., to another website). It also makes styling easier, you just have to set text-decoration: none to get rid of the underline!

  1. Finally, regarding accessibility, when you tab through the social links, you should use the focus pseudo-selector to style it and make it clearer which link you are currently on. For example:
.social-link:focus {
   background-color: // whatever color
}

Marked as helpful

1

P

@josifermaodev

Posted

@Carson-Haskell Thank you very much, your explanation was very good!

0
P

@Carson-Haskell

Posted

Happy to hear that! Keep up the amazing work :) @josifermaodev

1

Please log in to post a comment

Log in with GitHub
Discord logo

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