Responsive Order Summary Component

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted about 3 years ago

  👋Hi Carlton!

  👍 Good job on completing your first challenge! Some feedback from me:

  • The illustration hero image is stretched.
  • For any decorative images, like in this case the hero illustration and the music icon, you should leave the alt="" empty and add aria-hidden="true" or role="presentation" to make sure all screen readers ignore those images.
  • The Annual Plan should not be a heading. Heading tag is commonly used for titling a section or article. Use strong tag instead.
  • You don't need to specify, the height of the html element.
  • Use rem or sometimes em instead of px. Using px unit will not allow the user to control the size of your page.

  That's it! Hopefully this is helpful!

  1

  ---

  

  Carlton•130

  @CarltonMpofuPosted about 3 years ago

  @vanzasetia Thanks for the feedback!

  0