Skip to content
OverviewSolutions
1
Comments
1
Evan Lovett
@NotNotEvan

All comments

  • Evan Lovett•30
    @NotNotEvanPosted about 1 month 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