@AdamBtech
Posted
Hi guys, thank you for your feedback, i have updated the body and removed the margin on the container!
Would you have tips to make the code more DRY?
Thank you!
@AdamBtech
Posted
Hi guys, thank you for your feedback, i have updated the body and removed the margin on the container!
@danielmrz-dev
Posted
Hello @AdamBtech!
Your project looks great!
I have one suggestion:
š Don't ever use margin
with percentage values to move an element from the borders.
Here's a very efficient (and better) way to center the card:
body {
min-height: 100vh;
display: flex;
justify-content: center;
align-items: center;
}
I hope it helps!
Other than that, great job!
@tharidu678
Posted
you can set your body tag style like this..
.body{ display:flex; align-items:center; justify-content:center; height:100vh; }
this may help you to get rid off your scrollbar.
You only need one div in your body tag for this to work.. if you have one or two div s in body tag, put them in a div.
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