Resposive page using grid and flexbox

Community feedback

• 

  Adriano•38,080

  @AdrianoEscarabotePosted almost 2 years ago

  Hi Pedro Sena, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

  To improve the accessibility of the project you could have put an h1. Every page must contain a level 1 header, for people who use screen readers, identity what the main title is and follow the sequence h1-h5

  to solve this problem do this:

  <h1 id="name">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.

  <img src="product-preview-card-component-main/images/icon-cart.svg" alt="" aria-hidden="true">

  The rest is great!

  I hope it helps... 👍

  0