Design comparison
SolutionDesign
Community feedback
- @HassiaiPosted almost 2 years ago
Replace the <h3> with <p> to fix the accessibility issue.
Give the body the background-color and background-image you gave to the main.
For the left and right sides space of the page, give the main a max-width of 1440px, a width of 80% and margin: 0 auto.
main{ max-width: 1440px; width: 80%; margin: 0 auto; }
You forgot to add a media query for the mobile design. I will suggest you always start with mobile design first, it makes the process a little easier.
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful0@um1chc5Posted almost 2 years ago@Hassiai Thank you so much! I'm very appreciate your feedback. I'm going to fix these problem!
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