Four Card Feature Section using CSS Grid

Community feedback

• 

  T

  Grace•31,230

  @grace-snowPosted almost 4 years ago

  Nice job overall but it's broken for me on mobile. I think it's the large rem margin that's forcing a lot of horizontal scroll.

  Other small tweaks I'd recommend:

  • soften those card shadows to get closer to the design
  • fix the font weights
  • empty the alt attribute values on those icons. They're not meaningful images that need to be announced by assistive tech, they're just decorative

  I hope that helps you, good luck with this and your next challenge 👍

  1

  ---

  

  Binaya Bajracharya•295

  @binayab99Posted almost 4 years ago

  @grace-snow thanks for the feedback. I've updated it to make it look more closer to the design.

  1

  

  T

  Grace•31,230

  @grace-snowPosted almost 4 years ago

  @binayab99 nice one! It looks ace now, well done

  1