Order summary component

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted about 3 years ago

  Hey there, well done, your solution looks very nice! There are only a few small things to correct here:

  1. Giving a background-size: contain to the body would fix the image so it stretches across the screen nicely.

  2. min-height: 100vh on the body is needed for align-items: center to work properly and center your component vertically ^^

  3. You need to set a font-family on your button if you don't want it to use the default sans serif.

  4. Replacing your container div with main would be semantically more correct and would get rid of the issue in the report.

  1

  ---

  

  Joyal James•265

  @silkcoderPosted about 3 years ago

  @FluffyKas Thank you very much for your valuable comments. I corrected the issues you mentioned.

  0