All comments

• 

  P

  V.S Karthik Tirumalasetty• 50

  @VSKarthikT

  Submitted 8 days ago

  

  Solution using only grid

  1

  10

  

  Evan Lovett• 30

  @NotNotEvan

  Posted 8 days ago

  Hello!

  Overall, nice job on the challenge! A few things to consider:

  • Try to avoid hardcoded heights/widths - you could reduce a lot of your CSS code by using display: flex instead of a grid layout.
  • Avoid px for properties like font-size and letter-spacing - here's a great article on why. The BLUF is, using rem instead will respect the user's font preferences.
  • One small nit - You're missing some bottom padding on the card. Obviously not a huge deal, but if you're trying to go for the exact same design, I would recommend adding that.

  Great work. Keep going!

  • Evan

  Marked as helpful

  1