Andrés Gutiérrez RamÃrez• 2,530
@AGutierrezR
Posted
Hello!! It looks great!! I have a few suggestions that might be useful for you:
- Change the
.container
tag into a<main>
tag, and this<main>
tag should have amin-height: 100vh
instead of aheight: 800px
and of course ajustify-content: center;
to center the card. - If you set
justify-content: center;
on the.container
you could remove themargin: 20% auto;
in the.card
. - You could try to use custom properties for the colors, this will give you confidence when using variables in CSS.
Marked as helpful
0