Order Summary Component By Navid Dev

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi, David! 👋

  Good effort on this challenge! 👍 It's great that you are using the landmark elements correctly! 👏

  Some suggestions.

  • The b tag has been deprecated, instead I would recommend using strong instead. Note that not all bold text needs to be wrapped with strong.
  • Always use a meaningful element to wrap text content. I notice that you wrap the $59.99/year with a span, so I would suggest using the p tag instead.
  • For any decorative images, each img tag should have empty alt="" and aria-hidden="true" attributes to make all web assistive technologies such as screen reader ignore those images. In this case, all images are decorative only.
  • Also, alternative text for images should not contain any words that related to image (e.g. picture, photo, logo, icon, graphic, avatar, etc). It's already an image element so the screen reader will pronounce it as an image.

  I hope this information is useful! Happy coding! 😁

  Marked as helpful

  1

  ---

  

  Navid Dev•140

  @navidabdiPosted over 2 years ago

  @vanzasetia Thank you Vanza, I appreciate your effort to write this helpful comment. I'll decently consider this factors the next time :)

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @navidabdi No problem! Glad to help you! 😄

  1