@bccpadge
Posted
Hello there 👋🏼
I have a couple recommendations to improve your solution.
HTML
-
To make your website more accessible make sure you use a
<main>
tag instead of a<div>
tag -
Also I see you are using incorrect heading tag like a
<h4>
. Make sure you use heading tags in chronological order like a h1 through h6.
CSS
-
Be sure to check the styleguide.md file to get the correct colors to look like the design
-
To center your card component you can add
body{
min-height: 100vh;
}
you can remove the code below
body{
height: 100vh;
}
Hope this helps you!