Order Summary Component Solution

Community feedback

• 

  P

  Emmilie Estabillo•5,540

  @emestabilloPosted over 3 years ago

  Hey @trafiki, great work! Project is responsive and close to the design.

  • I would apply min-height: 100vh to the main tag so that the component spans the entire viewport height, and centers the component on the page (with flex). This way, you don't need the top and bottom margins on .order-card. The user will also be able to scroll down if the contents doesn't fit their viewport, better than height: 100vh

  • The alt text header-image doesn't convey meaningful content. I might just leave the alt attribute empty or use the image as a background

  • Good use of interactive elements. I would probably use an anchor tag for 'Proceed' since it usually brings the user to another page to complete payment

  Hope this helps!

  Marked as helpful

  3

  ---

  

  Akerele Tunde•165

  @trafikiPosted over 3 years ago

  Thank you so much @emestabillo. Thanks for min-height hack, I've always wanted that kind of behavior. This was really helpful

  1