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 Preview

ValafarL 170

@ValafarL

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@aweezzy

Posted

There are only a few things that I have queries about. So in your CSS you used flex-box in your main-card class. Well in some cases that may work, but in this project I believe grid will work just fine. It is strange the way you media queried the page, I got confused by reading the non queried style tags thinking they were the desktop page. The border radius is off. you added it to both your image and your "second card" which of course gave it the full top:right:left:bottom radius. Instead I would switch to grid for your main-card and create a grid-template of repeat(1fr) so that they evenly share the element and nothing overflows. Put only border radius: 0 5px 0 5px on the image and border-radius: 5px 0 5px 0 on the "second-card".

Marked as helpful

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