Skip to content
Submitted 10 months ago

Product Card Component

Velin Dankov•160
@IvelinDankov
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

So small component I did a lot of work. I use sass here. very carefully. Thank you about your feedback. We never give up! peace and love 👍

Community feedback

  • Daniel 🛸•44,250
    @danielmrz-devPosted 10 months ago

    Hello @Velingd!

    Your solution looks great!

    I have a couple of suggestions for improvement:

    • You set max-width: 144rem; to the container, but the background is getting cut off on both sides. If you remove it, you'll fix that.

    • Also, you don't need to use the tag article in this case. It would make more sense if the card was part of a bigger website (and it certainly would in real world) but here it's all we have.

    I hope it helps!

    Other than that, great job!

    0
    Velin Dankov•160
    @IvelinDankovPosted 10 months ago

    @danielmrz-dev Thank you, for your help. I will try better next time. That will help me a lot.

    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