@Islandstone89
Posted
HTML:
- Every webpage needs a
<main>
that wraps all of the content, except for<header>
andfooter>
. This is vital for accessibility, as it helps screen readers identify the "main" section of a page. Change.container
to a<main>
.
CSS:
-
Remember to include a good CSS Reset at the beginning.
-
As the card container is centered using Flexbox on the
body
, you don't needmargin: 0 auto
on.container
. -
max-width
should be on the card container,.card
. -
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. -
Remove
font-weight: 400
onbody
, as that is the default value. -
Ideally, you want to do mobile styling first, and use media queries(they should not be in px, but in rem) for larger screens.
Marked as helpful
@ayushyadavz
Posted
@Islandstone89 Review my latest challenge which I've done called Stats preview card.