Order summary component Solution #Flexbox #Media-Queries

Community feedback

• 

  Travolgi 🍕•31,300

  @denielden

  Posted over 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!

  01

  Marked as helpful
• 

  Zeynal Mardanli•180

  @Lshiroc

  Posted over 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/

  01

  Marked as helpful
• 

  Naveen Gumaste•10,420

  @NaveenGumaste

  Posted over 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!

  01