@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. Wrap.container
in a<main>
. -
"Learning" is not a
<button>
- buttons are used for triggering an action. Change it to a<p>
. -
Wrap the date in a
<time>
element:<time datetime="2023-12-21">21 Dec 2023</time>
. -
Headings should always be in order, so you never start with a
<h3>
. Change it into a<h1>
. -
The profile image needs a short and descriptive alt text.
-
Change "Gary Hooper" to a
<p>
.
CSS:
-
It's good practice to include a CSS Reset at the top.
-
Use the style guide to find the correct
font-family
. -
Add around
1rem
ofpadding
on thebody
, so the card doesn't touch the edges on small screens. -
Remove the margin on the card.
-
To center the card horizontally and vertically, use Flexbox on the body:
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
min-height: 100svh;
-
Remove the width on the card.
-
Add a
max-width
of around20rem
on the card, to prevent it from getting too wide on larger screens. -
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
opacity: 50%
, as it makes the text harder to read. -
Remove
float: left
. Instead, adddisplay: flex
andalign-items: center
on.comb
.
Marked as helpful