responsive landing page using CSS Grid

Community feedback

• 

  Adriano•38,330

  @AdrianoEscarabotePosted about 1 month ago

  Hi Giannis Mylonas, hope you're doing well! I loved how your project turned out, but I’ve got a few suggestions that could be useful:

  I noticed that you use more than one h1, this is not a good practice since it is recommended to have only one h1 per page to inform the main title of the page! so remove all h1 and put h2

  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 fantastic.

  Hopefully, you'll find it helpful. 👍

  0