Design comparison
SolutionDesign
Solution retrospective
I had difficulty making the background of the body, the image moves with the zoom, and I had difficulty because it came with two images.
Community feedback
- @WebDevMirzaPosted almost 2 years ago
Hi,
Congratulation on completing this project. You have already found a problem regarding bg-image.
HTML:
- Every html should have a
main
tag that you have not applied. Wrap everything inside<main></main>
to solve accessibility WARNING that you have now.main
is used for the unique part of the site where any repeated sections such as nav, sidebar, footer are not allowed. For more info visit W3 School
CSS 🙂
body { background: url(../img/bg-pattern-top.svg) no-repeat right 50vw bottom 40vh, url(../img/bg-pattern-bottom.svg) no-repeat left 50vw top 54vh; background-color: hsl(185, 75%, 39%); }
- This will add the background images at the position that they should be. 🧐
Apart from this, the rest of the part looks nice.👍
Marked as helpful0 - Every html should have a
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