Design comparison
Solution retrospective
Need more knowledge of twists and turns for improvement. Comments would be highly appreciated. Thank you in advance.
Community feedback
- @HassiaiPosted over 1 year ago
Replace <div class="wrapper"> with the main tag and <h3> with <h1> to fix the accessibility issues. click here for more on web-accessibility and semantic html
increase the max-width value of .wrapper.
max-width: 27rem which is 432px
.To center .wrapper on the page using flexbox or grid instead of a margin, add min-height:100vh; display: flex; align-items: center: justify-content: center; or min-height:100vh; display: grid place-items: center to the body.
USING FLEXBOX: body{ min-height: 100vh; display: flex; align-items: center; justify-content: center; }
USING GRID: body{ min-height: 100vh; display: grid; place-items: center; }
This challenge requires a media query in the media query you only have to change the background-image of the body. For more on media queries, click here
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
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