Social Links Profile

Solution retrospective
I think I got the sizing done much better using measuring tools online.
What challenges did you encounter, and how did you overcome them?Centering all items within the card and getting special spacing between some elements but not others. I was able to do this by using flexbox for the card element but applying data attributes to elements that had unique aspects.
I also used custom properties to set up reusable variables.
What specific areas of your project would you like help with?Be critical! What's a better way I could do something? What did I miss?
Please log in to post a comment
Log in with GitHubCommunity feedback
- @sunnyegg
Hi! First of all, good job on completing the challenge! There are some things that I noticed when I looked at your site and the code.
-
The website is fine when I open it on Desktop Screen, but when I downsize the screen to Mobile, the card got really tiny. I think you can just remove the
width
and theheight
on yourcard
class. Or you can set the maximum of the card's size usingmax-width
andmax-height
attributes. -
Remove unused codes in your CSS file.
Cheers!
Marked as helpful -
- @dbachour86
Nice work, but isn't it better to use
h2
,h3
, andul
tags? You are using a lot ofp
tags.
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