Design comparison
SolutionDesign
Solution retrospective
Any feedback will be great
Community feedback
- @HassiaiPosted almost 2 years ago
Replace <div class="logo"> with a header tag to fix the accessibility issue. click here for more on web-accessibility
<div class="logos"> should be within the main tag. wrap container01 and container02 and a section tag. the main out line :<main> <section> <div class= "container01"></div> <div class= "container02"></div> </section> <div class="logos"> </div> </main>
in the ccs give the section display: grid and grid-template columns value. give the grid template-columns 1fr and 25rem instead of1fr 1fr
Give the header, main and footer
max-width:1440px; width 80%; margin: 0 auto;
for the left and right sides space of the content. Add background-size of cover to the body.Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful1
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