order-summary-component-main

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 2 years ago

  Hi, Lasha! 👋

  Great work on this challenge! 👍

  A few suggestions I have are,

  • Adding padding to the inside of the container that wraps around the text content of the component rather than adding margin to the individual elements inside it. By specifying the spacing inside the container in one place, you'll make it easier for yourself to keep the spacing consistent around all of those child elements. 😉
  • Leaving the alt text for the image blank so that it will be ignored by screen readers (since it isn't very important to the content of the page and doesn't need to be read by screen readers).
  • Using a button element for the "Cancel order" element near the bottom of the component (since that element should be interactive).
  • Adding favicon to the site. There should be one in the free starter files for this challenge that you can use.

  Hope you find these suggestions helpful. 😊

  Keep coding (and happy coding, too)! 😁

  1