@Islandstone89
Posted
HTML:
-
Headings should always be in order, so you never start with a
<h3>
. Change it into a<h1>
. -
I see the links as an unordered list.
CSS:
-
Always include a CSS Reset at the top.
-
You have declared 0 padding and margin on
*
twice. -
Move
font-family
tobody
. -
Remove
width
onbody
- it is 100% wide by default. -
Remove the width on the card. you should rarely set fixed sizes. Instead, give it a
max-width
in rem. -
font-size
must never be in px. This is a big accessibility issue, as it prevents the font size from scaling with the user's default setting in the browser. Use rem instead.