Design comparison
SolutionDesign
Solution retrospective
Hello there 👋, I am @rajkp10
It is a good challenge for those who are still new to HTML and CSS and wants to learn by practicing.
The solution was implemented using HTML5 and CSS3 Flexbox.
Any kind of suggestion regarding the code or the design are most welcome. 🙂
Community feedback
- @0xabdulkhaliqPosted over 1 year ago
Hello there 👋. Congratulations on successfully completing the challenge! 🎉
- I have other recommendations regarding your code that I believe will be of great interest to you.
COMPONENT MEASUREMENTS 📐:
- The
width: 100vw
property for.container
is not necessary. because it's a block level element which will take the full width of the page by default.
- So feel free to remove
width: 100vw
style rule from.container
this will help you to write efficient code and makes your code more reusable.
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
Marked as helpful1 - @francescomerighi1202Posted over 1 year ago
Hi there!
Your solution is great!
Just a tip, avoid setting fixed heights whenever possible, let the card adapt to the content on its own!
Marked as helpful0
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