Design comparison
SolutionDesign
Solution retrospective
blog preview card
Community feedback
- @BlackpachamamePosted 9 months ago
Greetings! you have done a great job 😎
- To improve the semantics of your HTML, you can change your
<div class="content">
to a<main class="content">
- Apply
max-width: 100%
to yourimg
so that it occupies the correct width within the container - Instead of using
margin
to center your content in the center of the screen, you can use theflexbox
properties in thebody
(Remove the margin from your content class):
body { background-color: hsl(47, 88%, 63%); min-height: 100vh; display: flex; justify-content: center; align-items: center; }
Marked as helpful0@maleyka13Posted 9 months agoHi @Blackpachamame, thank you for your comment. I will check this
1 - To improve the semantics of your HTML, you can change your
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