Skip to content
Submitted 5 months ago

Social links profile

Masseh•210
@Masseh2025
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?

Im very proud about how close the solution looked liked to the real one even thought I didn't use a figma file. One thing I would do differently would be to name my classes better.

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

The challenges I encountered were that the links kind of looked off in the padding, I overcame them by trying different solutions intill I found a padding that worked.

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

I dont know specificly but any feedback would be fine.

Community feedback

  • Dylan de Bruijn•3,230
    @DylandeBruijnPosted 5 months ago

    Hi @Masseh2025,

    Congratulations on a great solution, it looks very good! You have an eye for design. Good use of CSS variables and relative CSS units like rem.

    A bit of friendly constructive feedback:

    • You could try using more semantic tags like a h1 tag instead of a div for the author's name. This helps screen reader users to navigate your page.

    • Instead of border-radius: 1000px you could use border-radius: 50% to turn your image in a circle.

    • I would suggest putting your a tags in the li tags in the list to keep the structure intact.

    I hope you find my feedback helpful and if you have any other questions let me know!

    Marked as helpful
    1
    Masseh•210
    @Masseh2025Posted 5 months ago

    Thank you for the feedback! I will keep that in mind.

    0

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