@mkboris
Posted
Hi Stroudy this looks great, you can make it better by adding this lines of code on the body so that the card is properly centered. Doing this, you wouldn't need to use margins on the card to center it, you can remove the margins once this is set on the body.
display: flex;
align-items: center;
justify-content: center;
min-height: 100vh;
Also, remove the height
from the card completely and change the width
to max-width
. Avoid fixed heights and widths as they can hinder layout adaptability and content flow. Instead, allow padding and content to determine element sizes. Use max-width
and min-height
only when essential.
Marked as helpful
@Stroudy
Posted
@mkboris Hello, thank you for your feedback i have made the correct adjustments, I did run into a issue where margin top was causing the 100vh not to work as intended but i have done it. Apricate you.