Order Summary Component

Community feedback

• 

  David•8,000

  @DavidMorgadePosted about 2 years ago

  Hello Nien congrats on getting the solution to the challenge!

  If you don't mind I would like to give you some feedback.

  First of all, instead of using justify-content: space-between on your body, try using center instead to get the card in the middle of the screen.

  The background image seem to not display fully screen on higher resolutions, try using background-size: contain; to get it fixed!

  Apart from that, everything seems to be working as intended, great job in your CSS!

  Hope my feedback helps you!

  Marked as helpful

  1

  ---

  

  Nien•170

  @trandainienPosted about 2 years ago

  @DavidMorgade Thank you so much for the comment. i learned something new from you, really appreciate it. <3

  1