Design comparison
Community feedback
- @Islandstone89Posted 10 months ago
HTML:
-
The icons are decorative, so the alt text should be empty:
alt=""
. -
"Learn More" would navigate to another page, hence it is not a button but a link.
-
Footer text must be wrapped in a
<p>
.
CSS:
-
It's good practice to include a CSS Reset at the top.
-
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. -
You have written
min-height: 100svh
twice. -
Again, only declare
max-width
on the card once. Also, it should be in rem. -
Media queries should be in rem, not
px
.It is common practice to do mobile styles as the edfault, and use media queries for larger screens.
0 -
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord