Submitted over 3 years ago
3-Column preview card component with plane CSS and HTML
@surphury
Design comparison
SolutionDesign
Solution retrospective
Please feel free to amend anything in the code.
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hey there, Jose Berrios! 👋
Good job on this challenge! 👏 Your solution looks great and is responsive! 👍
I suggest,
- Adding a little space around the top/bottom of the card component in the mobile layout so that it isn't right up against the edges of the screen.
- Adding
cursor: pointer
to the buttons in the card (or even making them links instead, since they seem like they might take the user somewhere when clicked). - Using a heading tag (such as
<h2>
or<h3>
) for the titles of the sections of the card (since that is more fitting for headings, and the<strong>
doesn't need to be used for that). 😉
I hope those few tips help. 🙂
Keep coding (and happy coding, too)! 😁
Marked as helpful1@surphuryPosted over 3 years ago@ApplePieGiraffe Thank you! I'll do it, I really appreciate your feedback
1 - @Amanpatil-DevPosted over 3 years ago
Nice work Looks good 👌👍
Marked as helpful1@surphuryPosted over 3 years ago@Amanpatil-Dev Thanks, I really appreciate your words.
1
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