@FluffyKas
Posted
Hey, it's a good start but there's a few things you could do!
-
Your solution doesn't seem to be responsive. Perhaps if you set the container's width to max-width instead, that would help.
-
Your need to set a font-family for the button, otherwise it will use the browser's default. Adding a
cursor: pointer
would be a nice touch too! -
There's an overflow on your body that comes from setting fixed heights. There's usually no need to set heights on your elements, except your body and even there, a min-height of 100vh would be the best. If you add
display: grid
andplace-items: center
that would nicely center your component. -
4.8rem of padding everywhere on your content is a bit of an overkill. If you get rid of the fixed height, you'll not need to have such a large padding everywhere, you'll see.
Marked as helpful