3-column preview card solution | scss, grid

Community feedback

• 

  Abhijit Barman•550

  @abhijitbcobPosted over 3 years ago

  looking very nice! Here is my some tips:

  • to center any element in body just only use display: grid; place-items: center;
  • instead of writing row height, use padding inside every card component.
  • in mobile view write grid-template-columns: minmax(0, 325px) so that the card be responsive.

  Marked as helpful

  0

  ---

  

  Gutka•65

  @karbowskamPosted over 3 years ago

  @abhijitbcob thank you for your advices :)

  0
• 

  ToNisamBioJa•330

  @ToNisamBioJaPosted over 3 years ago

  You indeed are on fire!

  One small thing is that you forgot to put border radius on your boxes.

  Good work here. Keep it up and happy codding!

  Marked as helpful

  0

  ---

  

  Gutka•65

  @karbowskamPosted over 3 years ago

  @ToNisamBioJa Oh yeah, border radius. I forgot about it completely. I'm going to fix it. Thank you so much for your feedback :)

  0