Four Card Feature with CSS Grid

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi, Nwobodo Igwe! 👋

  Good effort on this challenge! 👍 Your site looks great on my desktop view. 😄

  I have a few suggestions for this solution.

  • I would recommend adding some padding to the body element to prevent the elements from touching the browser edges. 😉
  • I suggest making the below HTML markup as one h1 instead of two. You can wrap the second text with span and then make it as a block element. This way the text would move to the next line.

  <h1>Reliable, efficient delivery</h1>
  <h1>Powered by Technology</h1>
  

  • For any decorative images, each img tag should have empty alt="" and aria-hidden="true" attributes to make screen readers ignore the image. In this case, all images are purely decorative.

  I hope this helps! Keep up the good work! 👍

  1
• 

  Raj Prasad Sanjel•980

  @RajSanjelPosted over 2 years ago

  My suggestion

  • Use semantic html tags Read here
  • Make breakpoint of 1100px width since there is not spacing in right and left and this ruins the user experience and design looks kinda ugly.

  1