Skip to content
Submitted 23 days ago

Social Links Profile

P
Selina Lavery•50
@SelinaLaveryDev
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 didn't set any fixed height and width with the components and tried using padding instead. Was more of a challenge

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

I struggled with the font-weights when using @font-face so ended up downloading from google fonts which then worked.

Community feedback

  • P
    laurice-dev•160
    @laurice-devPosted 23 days ago

    Hi Selina,

    I loved that you used variables within your code, kudos! Just a couple of mini notes for consideration. CSS: When selecting an [img] it's good to define the height and a max-width. Example: height: 88px; max-width: 88px; This will allow for the image to be more responsive. Also, border-radius: 50%; would be easier for future images.

    I saw that you declared a "font-family" on .text-preset-1, .text-preset-2, .text-preset-2-bold and this could be omitted because you've already declared a font-family for the body element. If the font-family was different from the body selector, then it would be good to declare it there.

    Kindly try to use rem instead of px, if possible. :-)

    You're doing amazing and we're all in this together. Keep up the great work.

    Marked as helpful
    1
  • P
    Carlos Eduardo Santos Oliveira•180
    @Carlos-Eduardo-SPosted 23 days ago

    Muito bom!! Parábens.

    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