Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social Link Profile

@GabrielOliveiraEvangelista

Desktop design screenshot for the Social links profile coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Daniel 130

@tenczowy

Posted

Consider using more semantic elements where appropriate. For example, you could use <section> instead of <div id="main"> and <article> for the header section.

Instead of using border-radius: 9999rem; to make circle out of picture you can use border-radius: 50%;

Also you are styling img in 2 places, id=img-box and img itself. You could do same thing by not having div for img at all and just styling img and adding some margin.

img{ max-width: 25%; display: block; border-radius: 50%; margin: 2rem auto; }

0

Please log in to post a comment

Log in with GitHub
Discord logo

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