Submitted about 1 year ago
3-column preview card using mobile-first
@abhishek-baliyan-dev
Design comparison
SolutionDesign
Community feedback
- @Islandstone89Posted about 1 year ago
Hi, good job! You have done several things most people miss:
- The
<main>
element, which is important for accessibility. - Empty alt text on what is decorative icons.
- Font-size in rem instead of px.
- Using
min-height
instead ofheight
on the body.
Here are some tips to improve your solution even further:
HTML:
- "Learn More" would navigate to another page, hence it must be a link.
CSS:
-
Remove the fixed width on the <main>. Add a
max-width
in rem. -
Media query should also be in rem.
Marked as helpful1 - The
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