Product preview card component

Community feedback

• 

  Adriano•42,890

  @AdrianoEscarabote

  Posted 10 months ago

  Hi Kairo, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

  Since this project is only based on a single page component, there is no need for a h1 tag. It's always a good idea to prevent accessibility errors, so I believe it would be beneficial for you to add a "h1" in this component. Don't worry if you forget about "h1," though; it's a good practice for when you are developing larger sites.

  <h1>Gabrielle Essence Eau De Parfum</h1>
  

  images must have alt text unless it is a decorative image, for any decorative image each IMG tag must have empty alt="" and add aria-hidden="true" attributes to make all the assistive technologies of the Web, as screen reader.

  Learn the differences between decorative/meaningless images vs important content.

  The rest is excellent.

  I hope you find it useful. 👍

  11

  Marked as helpful
• 

  Tonyac-create•220

  @Tonyac-create

  Posted 10 months ago

  You don't use a good semantic. You can use balise section, article, header and footer.

  03