Design comparison
SolutionDesign
Community feedback
- @IzykGitPosted 3 months ago
Good job but remember to maintain SEO!
So instead of using a div as your main container, make it a <main> element. As well as have the name of the person be a <h1> element and the location be a <h2> element for example.
Divs do no have any semantic value so the browser won't know what is the main content of the page, hurting SEO!
Also to simply center a container you can do:
display: flex; align-items: center; justify-content: center;
Keep up the good work. :)
Marked as helpful0
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