This was my second css grid

Community feedback

• 

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted about 2 years ago

  Hey Troy! It's great to see that you have uploaded yet another awesome solution. Your report is looking so clean, kudos! 👌 Anyways It's responsiveness is consistent on all screens but I think it'd be a lot better if you switched to one grid column at a minimum screen width of 650px because your card component still fits in the layout even at that screen width. (I also recommend using pixels instead of em or rem to specify screen widths in your media queries)

  • You might also want to add a hover state to your button to give it a more interactive feel and make it look less static. Otherwise, this is a very solid solution, keep up the great work and keep on coding! 👍

  Marked as helpful

  1

  ---

  

  Troy•170

  @troy03Posted about 2 years ago

  @tesla-ambassador i watch a tutorial explaining why use of em in media queries hahaha thanks up to now I'm studying sass and creating a project using css precessor.

  1

  

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted about 2 years ago

  @troy03 Great work Troy, keep it up! I also enjoy using sass, it makes writing css a lot more easier.

  1