Social Links Profile with Simple Responsiveness

Solution retrospective
I am pretty proud of this design. Because I used rem
instead of px
in this design. And also instead of media queries, I used max-width
.
Nothing really. It was a smooth experience.
What specific areas of your project would you like help with?I can't think of any. But I am open to any recommendations and feedbacks.
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@Islandstone89
Hi, well done!
A couple of tips to improve your solution:
HTML:
-
Instead of using
<section>
for the card, I would consider the<article>
element. You could also use just a<div>
. -
A better alt text for the image would be something like "Headshot of Jessica Randall".
-
"London, United Kingdom" is not a heading but a
<p>
.
CSS:
-
Make a habit of including a modern CSS Reset at the top of the stylesheet.
-
Move
font-family
andcolor
tobody
. Except forpadding
,margin
andbox-sizing
, we rarely set styles on the universal selector*
. -
I recommend adding a bit of
padding
, for example16px
, on thebody
, to ensure the card doesn't touch the edges on small screens. -
Descendant selectors like
.social-links-list li
and.social-links-list li a
increase specificity, making the styles harder to override. Instead, give elements a class and use that as the selector. -
max-width
on the card should be in rem. Around20rem
works well.
Marked as helpful -
- @Mirouf18
Good job !
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