Design comparison
SolutionDesign
Community feedback
- @tenczowyPosted 10 months ago
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 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