Submitted almost 2 years ago
3-column Preview Card Component with HTML, CSS
@ElizabethKennedy
Design comparison
SolutionDesign
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- To properly center your content to your page, you will want to add the following to your
Body
element (this method uses CSS Grid):
body { min-height: 100vh; display: grid; place-content: center; }
More Info:📚
- The “car images/icons” in this component are purely decorative; They add no value. So their
Alt Tag
should be left blank and have anaria-hidden=“true”
to hide them from assistive technology.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- Your
attribution
should be wrapped in afooter
element instead of adiv
.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
Marked as helpful0@ElizabethKennedyPosted almost 2 years ago@vcarames, thank you for the helpful feedback, I appreciate it! I implemented all of your suggestions, which noticeably improved the quality of my solution. Thanks again!
1@VCaramesPosted almost 2 years ago@ElizabethKennedy
Glad I could help!☃️
Keep it up!
1 - To properly center your content to your page, you will want to add the following to your
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