@Islandstone89
Posted
Hi, well done!
Here are some suggestions:
HTML:
-
Don't include "image" in the alt text, as screen readers announce it as an image automatically.
-
Wrap the date in a
<time>
element:<time datetime="2023-12-21">21 Dec 20223</time>
. -
For the profile image, an alt text like "Headshot of Gary Hooper" would be more suitable.
CSS:
-
Performance-wise, it's better to link fonts in the
<head>
of the HTML than using@import
. -
It's good practice to include a CSS Reset at the top.
-
Add around
1rem
ofpadding
on thebody
, so the card doesn't touch the edges on small screens. -
max-width
on the card should be in rem. -
font-size
must never be in px. This is bad for accessibility, as it prevents the font size from scaling with the user's default setting in the browser. Use rem instead. -
Media queries must be in rem. Also, it is common to do the mobile styling as the default.
Marked as helpful