Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Product Card Component

@IvelinDankov

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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,230

@danielmrz-dev

Posted

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

@IvelinDankov

Posted

@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
Discord logo

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