3-column preview card component

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Rosalia, congratulations for your solution!

  I saw your live site and is really good done, all the design elements are perfect.

  My advice for you is to set the media query a little later and let the vertical container contract a little bit more or let the container grow almost 100% of the screen when the media query changes`, you can note that there's a big big space between the card and the screen borders when the component change the flow vertically.

  There's also no need to set a fixed height for each column like .sedan-column, .suv-column, .luxury-column the column will grow only with the elements and its paddings/margins, if you set a fixed height this can make the content get cropped when the container start to scales and overlap the fix height.

  Marked as helpful

  0

  ---

  

  Rosalia•90

  @RosaliaLeyPosted over 2 years ago

  @correlucas Thank you so much!

  1
• 

  faizan•2,420

  @afaiz-spacePosted over 2 years ago

  Hey @RosaliaLey,

  • Remove margin: 0 0 0 2em; from the .sedan-column, .suv-column, .luxury-column class.

  Marked as helpful

  0

  ---

  

  Rosalia•90

  @RosaliaLeyPosted over 2 years ago

  @afaiz-space Thank you Faizan! Completely forgot that there.

  0
• 

  Adam M•550

  @AdamMzkrPosted over 2 years ago

  Great job but, you can add a little bit bottom padding in each column in mobile view that's in my opinion will looks better 😉

  0

  ---

  

  Rosalia•90

  @RosaliaLeyPosted over 2 years ago

  @AdamMzkr Thank you :) You're right!

  0