Responsive landing page using CSS Grid and Flexbox

Community feedback

• 

  Adriano•38,410

  @AdrianoEscarabotePosted about 2 years ago

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

  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.

  I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written Read more, he is not confirming a form, or something like, it will be redirected to another page, to read more about!

  To resolve do this:

  <a href="/" class="grid1-img-btn">READ MORE</a>

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  0