Order Summary solution

Community feedback

• 

  Harsh Porwal•30

  @mysteriousplaPosted about 3 years ago

  Thank Jason Moody for providing me such valuable feedback.

  0
• 

  Jason Moody•300

  @MoodyJWPosted over 3 years ago

  Hey there! I pulled your repo and it looks like you can center the footer text by just removing the margin you've placed on it.

  .code {
     text-align: center;
     ~~display: flexbox;~~
     ~~margin-right: 100px;~~
  }
  

  I'd also like to give you a couple of pointers. You are using quite a few CSS properties with invalid values.

  • padding can't be set to auto (margin can, though!)
  • flexbox as the value for the display property, but the valid value is simply flex or inline-flex
  • font-weight can't be set to a unit using rem, you only need a number or a word. If you want a font-weight of 900, font-weight: 900 is all you need!

  I'd also suggest learning more about using flex-box as it would make your CSS much easier to manage. The way you've set the margins to center the card works (sort of), but as soon as the screen size changes it will be in the wrong position. Flexbox isn't really something I can break down in a comment, but this video series is pretty good:

  Youtube - Kevin Powell, Flexbox Basics

  And of course, the MDN usually has everything in writing:

  0