@kaamiik
Posted
Hi. Some notes:
-
footer
should be outside the main as a sibling not a child. -
img
is decorative and no need to alt text and it should be empty likealt=""
-
Use
min-height:100vh
instead ofheight
-
For your
.badge
usep
tag that is more semantic. -
Your font-size should never be in
px
unit and you have to userem
instead. -
Your
h2
is interactive because hashover
effect. when an element has hover effect, then it should be interactive. So you have to wrap it betweena
orbutton
too. If It takes you to a new page then It'sa
and If doing an action then It'sbutton
. -
Use a proper CSS reset for your styles. Andy Bell has a good one.