3 Column Preview Card

Community feedback

• 

  hardy•3,660

  @hardy333Posted about 3 years ago

  Hey, nice job, card look good.

  I suggest not to use too many media query break points, they are not that necessary. Imagine when you have very complex design and way more components maintaining that many breakpoints will be just impossible. So my suggestion will be to use only one breakpoint - lets say on 800px wide or so and then make only one column. Or you can even don't use media queries at all and use flex-wrap: wrap; property; it depends on your preference.

  Marked as helpful

  3
• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted about 3 years ago

  Hiya, you could also do this on the main-container: border-radius: 10px, overflow: hidden. The reason you couldn't see them before is probably because the background color from the individual cards are overflowing. You also seem to be missing alt texts (all images must have them, in this case, since they're decorative, you can just leave them empty). Apart from this, it really looks good, well done!

  Marked as helpful

  0
• 

  darryncodes•6,370

  @darryncodesPosted about 3 years ago

  Hi Shawn,

  Nice work - well done!

  You could add the following to the relevant classes and corners of the boxes (don't forget to update them on mobile view too):

  border-top-left-radius: 10px;
  border-top-right-radius: 10px;
  border-bottom-right-radius: 10px;
  border-bottom-left-radius: 10px;
  

  Marked as helpful

  0