Skip to content
Submitted over 1 year ago

Responsive product card component using grid and flexbox

geektim•230
@geektim
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Product preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I'll have to redo this challenge, as i did it before the learning paths feature came on board

What challenges did you encounter, and how did you overcome them?

Scalability

What specific areas of your project would you like help with?

Feedbacks, please!

Community feedback

  • Akshay minz•320
    @AkshayV30Posted about 1 year ago
    • yes the solution is semantic HTML.
    • yes it is
    • It looks good
    • yes, the code is well-structured, readable, and reusable
    • yes a bit
    1
  • P
    Stefan Holmgren•220
    @stefan-holmgrenPosted about 1 year ago

    Looking good!

    Only visual issue I saw was a white border to the left and right of the page (I have a wide monitor), and that's probably because .container has the background-color, not the body.

    Other than that, I notice you use px alot. I've had people/sites/youtube tell me to steer clear of that and go with rem, and I try to do that

    https://www.seeratawan.me/blog/why-you-should-move-from-px-to-rem-in-css/

    1

Please log in to post a comment

Log in with GitHub

Join 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