Product card made with flexbox and preprocessor sass

Community feedback

• 

  Gustavo de Camargo Campos•200

  @themegazordPosted over 2 years ago

  Hi Kamil, how are you?

  Another thing I noticed, you inserted the mobile image in the desktop version.

  Marked as helpful

  1

  ---

  

  Kamil•120

  @kamilp522Posted over 2 years ago

  @themegazord Done, thanks for the tip!

  0
• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Kamil, congratulations for your solution!

  I saw your live site and the component border-radius are just perfect, no issues there. What was the problem?

  The only issue I see is that the image isn't resizing with the container with the mobile version. You can fix that using the following properties:

  img {
  display: block; 
  max-width: 100%;
  object-fit: cover; 
  }
  

  Hope it helps and happy coding!

  Marked as helpful

  0

  ---

  

  Kamil•120

  @kamilp522Posted over 2 years ago

  @correlucas Hi Lucas! Thanks for the feedback and the tip on the img element! I actually couldn't add border radius to the card container so I've added it to the image wrapper and descriptions container separately. Otherwise it couldn't work, and looked like this: https://postimg.cc/mhcDTQrP . It kinda works but content goes over the border.

  0