Submitted 3 months ago

Social links profile

Masseh• 210

@Masseh2025

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Social links profile

HTML
CSS

1newbie

View challenge

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

@DylandeBruijn

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

Masseh• 210

@Masseh2025

Posted 3 months ago

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

Please log in to post a comment

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