
Design comparison
Community feedback
- @thurgarion2Posted 3 months ago
Overall good job! I think you don't need to wrap the links in paragraphs. You can directly apply the social-link" style to the links. You can also remove the div with the "text" class, as it does not apply new styles to its children.
0 - @brodel10Posted 3 months ago
Amazing job Ayman, your solution looks exactly like the design. I would recommend using max-width and max-height though as modern browsers detect this and keeps a correct aspect ratio and makes it responsive across all screen sizes.
0 - P@Bobstyle23Posted 3 months ago
Good job! Would be better if you increase spacing between image and name and decrease a little bit of space between profile links
0
Please log in to post a comment
Log in with GitHubJoin 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