@DylandeBruijn
Posted
@charukirti
Hiya! 👋
Congratulations on your solution, it looks very close to the design! I can tell you put a lot of effort into it.
Things I like about your solution 🎉
- Responsive
- Use of semantic HTML elements
- Clear descriptive CSS classes
- Use of CSS custom properties
Things you could improve ✍️
-
Add a bit of
padding
to yourbody
element so the card has room to breath on smaller viewports. -
Add
href
attributes to your links so they have the default styling including the pointer cursor. -
Remove
flex-direction: column
on thebody
element. This property is unnecessary at the moment. -
Try experimenting more with Flexbox.
I hope you find my feedback valuable, and I would appreciate it greatly if you could mark my comment as helpful if it was! 🌟
Let me know if you have more questions and I'll do my best to answer them. 🙋♂️
Happy coding! 😎
Marked as helpful