Design comparison
SolutionDesign
Solution retrospective
Please give me notes ...
Community feedback
- @JorgixhalloPosted almost 3 years ago
Very good! Like me, you also forgot the box shadow. It's not very noticeable, but will add a good detail to the final result. As an additional detail, I would not limit the width ("width: 1440px;") and make would make height 100vh because your current solution will cut the background if your windows is bigger than 1440px.
1 - @caio-alcantaraPosted almost 3 years ago
Hey, it looks great. To fix those accessibility issues you just need to wrap the body content into the tag <main>
1
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