3 Column Card Component using CSS Grid

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 2 years ago

  Hello, Vitor Aragone! 👋

  Great work on this challenge! 👍

  A couple of things I'd like to suggest are,

  • Leaving the alt text for the icons in the card component empty. That will cause screen readers to ignore those images since they are not important to the content of the page and do not need to be read by the screen readers.
  • Making sure to add some space between the top and bottom of the card component in the mobile view of the site.
  • Using only one h1 tag per page. This is a good practice since there should be only one most important heading on a page. You can use less-important headings tags (such as h2 and h3) for the headings in this card component.
  • Switching to a the mobile view slightly sooner in order to avoid having a horizontal scroll bar appear along the bottom of the page. 🙂

  Hope this helps. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  1

  ---

  

  Vitor Aragone•160

  @vitoraragonePosted almost 2 years ago

  @ApplePieGiraffe really thanks for the suggestions! They will be very helpful 😊

  1