Submitted 20 days ago

Social links profile

Fatma Tawfeek• 100

@Fatma-Tawfeek

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

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?

organization of the code
using semantic HTML

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

make "ul" element take the 100% of the card using max-width
I used the width property instead of max-width

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

adjust spaces between text and other elements

Community feedback

tagoras• 50

@tagoras

Posted 20 days ago

I would suggest the following changes:

Increase the overall size of the card, including font-size, margins and paddings. You can do this by setting html{font-size: 62.5%} and then setting the font-size to an appropriate amount in your card container using rem units. Same with margins and paddings.
Decrease the margin between the person's title and the location.
Increase margin of links, make the name of each social link bold (font-weight: bold or using an appropriate numerical amount).

Nice solution!

Marked as helpful

Fatma Tawfeek• 100

@Fatma-Tawfeek

Posted 19 days ago

@tagoras Thank you for your feedback! I'll definitely keep it in mind for next time.

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