Order-summary-component-main

Community feedback

• 

  Olaniyi Ezekiel•7,580

  @Ezekiel225Posted 10 months ago

  Hello there 👋.

  Good job on completing the challenge !

  Your project looks really good!

  I have suggestion about your code that might interest you.

  Consider changing the background-color of the body element to hsl(225, 100%, 94%) so as to match the original design.

  body {
    background-color: hsl(225, 100%, 94%);
  }
  

  I hope it helps!

  Other than that, great job!

  Happy coding.

  Marked as helpful

  0
• 

  Daniel 🛸•44,270

  @danielmrz-devPosted 10 months ago

  Hello @tariqauwal2020!

  Your project looks great!

  I noticed that you used margin to place the card in the middle of the page. Here's a very efficient (and better) way to center the card:

  • Apply this to the body (in order to work properly, don't use position or margins):

  body {
      min-height: 100vh;
      display: flex;
      justify-content: center;
      align-items: center;
  }
  

  I hope it helps!

  Other than that, great job!

  Marked as helpful

  0