Order-summary-component

Community feedback

• 

  Travolgi 🍕•31,300

  @denielden

  Posted about 3 years ago

  Hi Marianpicco, great work on this challenge! 😉

  Here are a few tips for improve your code:

  • the link of google font go inside the head of document not into body
  • for fix the top image in the background just put more specific background properties to the body:

  background: url("../img/pattern-background-desktop.svg") no-repeat top center;
  background-size: contain;
  background-color: #e0e8ff;
  

  • add margin:0 to body because with flex they are superfluous
  • after, add min-height: 100vh to body because Flexbox aligns child items to the size of the parent container

  Overall you did well 😁 Hope this help!

  01

  Marked as helpful
• 

  Bhremada Fevreano•1,395

  @boedegoat

  Posted about 3 years ago

  Good job @Marianodsad

  It seems like the background images are repeating which is not very good. Here's how you can fix that :

  background-repeat: no-repeat;
  

  Add that property and the problem should be fixed.

  Other than that, keep learning and enjoy your coding journey ☕

  00

  Marked as helpful