Order summary component Solution #Flexbox #Media-Queries

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hi Nico, great job but the paper is so big that you can't see it all and it is cut off the screen.

  • The font-size is so much big... reduce it at 1rem for the body and 1.5rem fot the h1
  • for the internal spaces of the containers of the elements try to use the padding instead of the margin
  • remove width: 40% from card class.
  • remove margin from .card div
  • You just miss them the effect :hover

  Also in the mobile design adding padding:1 rem to the body.

  I hope that helps ;) Keep it up!

  Marked as helpful

  0

  ---

  

  Nico Radin•40

  @niicodeerPosted almost 3 years ago

  @denielden thank you! I will fix it :D

  1
• 

  Zeynal Mardanli•180

  @LshirocPosted almost 3 years ago

  Hi👋, I think you should not change the font-size because they already gave us the appropriate text font-size so we should use that. If you want to change the font-size while screen size changes, then I think you should take a look to fluid typography https://css-tricks.com/simplified-fluid-typography/ https://css-tricks.com/snippets/css/fluid-typography/

  Marked as helpful

  0

  ---

  

  Nico Radin•40

  @niicodeerPosted almost 3 years ago

  @Lshiroc thank you for the info!

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Good Job

  -> Try to set the card height and width so that it fits in center

  -> Check my solution for help

  Keep up the good work!

  0

  ---

  

  Nico Radin•40

  @niicodeerPosted almost 3 years ago

  @Crazimonk thank you!

  0