Four Card Feature Section / React / Sass

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 2 years ago

  Hello there, Jeisson Gómez! 👋

  Nice effort on this challenge! 👏 I like what you did with the feature cards in the tablet layout of this challenge! 👍

  A couple of things I suggest are,

  • Using an h1 tag for the main (and most important) heading of the page ("Reliable, efficient delivery, Powered by Technology").
  • Leaving the alt text for the icons in the feature cards empty (so that assistive technologies will skip over them since they aren't very important to the content of the page).
  • Horizontally centering the feature cards in the mobile view of the site.
  • Making sure none of the feature cards overlap each other in the tablet/mobile views of the site (you might want to remove the transforms on some of the feature cards for those layouts). 😉

  Hope you find this helpful. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  2

  ---

  

  Jeisson Gómez•120

  @jeissongomezdevPosted over 2 years ago

  Thank you very much @ApplePieGiraffe! Every point you made is clear to me. Thanks to your comments I can improve my knowledge. Cheers! 👍

  1