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

Responsive Product Preview Card

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


Kindly review and feedback. Thanks.

Community feedback

P
gfunk77 1,200

@gfunk77

Posted

Nice job on your solution. Your card looks great!

  • I see that you have multiple media queries but they are all defining the same thing. If your media query is @media (min-width: 1200px) {} all of your style can go in the curly braces. You don't need multiple media queries.

  • Since you represented your card as the <main></main>, I would suggest adding

body { display: flex; justify-content: center; align-items: center }

This will center your <main> on the screen and you don't need margins.

I hope this helps. Again, great job on completing the challenge!!

Marked as helpful

0

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