Design comparison
SolutionDesign
Community feedback
- @lisztomania23Posted about 1 year ago
Hello Obi, I looked at your code and saw that you centered the card by using
margin
. It is not good practice. Instead, you could have achieved it by giving thebody
aheight
of100vh
or100svh
anddisplay: flex;
flex-direction: column;
justify-content: center;
align-items: center;
and also for correctly positioning the background, instead of using:
background-position: left top, right bottom; background-size: 700px, 1000px; background-position-x: -10px, 600px; background-position-y: -350px, 300px;
you could have put
background-position: right 50vw bottom 45vh, left 50vw top 40vh;
Congrats on completing the challenge! Happy coding!
Ping me if you want me to make a few changes and create a PR.
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