HTML, CSS, Flexbox (Order summary card challenge)

Community feedback

• Account deleted

  Hello! Just wanted to point out that your media query doesn't seem to work. The problem is with this code you commented out:

  /* .cancel-href::before {
    ...
  }
  .cancel-href:hover::before {
    width: 100%; */
  }
  

  The curly bracket at the very bottom is not commented out, which prevents the media query code below it from working. You should either remove the commented code altogether, or close the comment out on that curly bracket.

  Besides that, nice work!

  Marked as helpful

  1

  ---

  

  Anna•140

  @AnnaUlianovaPosted about 3 years ago

  @idiotic-enigma Hello, yeah, indeed, i was trying to test some additional feature and being in a hurry missed that curly bracket, thanks a lot for noticing and your feedback :)

  0
• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted about 3 years ago

  Hey, a good looking solution! I'd have two suggestions though!

  1. Your background: it needs a background-size: contain to cover the whole screen. Problem is, you used an additional (and unnecessary) container div to center your component, and that's where your height is (which you could change to min-height) which is needed for your background image to work well. So you either apply the background image on that container div, or move all the styling to your body. The latter will result in less code, so I'd suggest doing that, but it's up to you of course.

  2. Your payment button uses the browser's default font-family, don't forget to change it! ^^

  Again, congrats on completing your first challenge, really good job ^^

  Marked as helpful

  0

  ---

  

  Anna•140

  @AnnaUlianovaPosted about 3 years ago

  @FluffyKas thank you, i found your suggestions very helpful, my work looks complete now ^^

  0
• 

  Nabil•160

  @Nabil19911Posted about 3 years ago

  Look amazing.

  Marked as helpful

  0