Order summary component using vanilla css

Community feedback

• 

  Mohamed Hussein•305

  @MohamedHussein05Posted over 3 years ago

  Great work on this one, I'll just Suggest few things to improve your work:

  1. The Card doesn't appear properly in the Landscape mode using body { (min-height: 100vh;) } instead of the current rule and removing #bottom-section { (height: 70%;) } will fix it.

  2. It's not recommended to set height on the card Use paddings & margins instead.

  3. The scrollbar isn't very nice, disabling it will give your page a better look.

  4. Try as hard as you can not to use height .. but If you must use min-height.

  Marked as helpful

  0

  ---

  

  jimavictor•375

  @jimavictorPosted over 3 years ago

  @MohamedHussein05 Thanks a lot for the feedback. i will make these changes as soon as i can..

  0
• 

  hardy•3,660

  @hardy333Posted about 3 years ago

  Nice work.

  4

  ---

  

  jimavictor•375

  @jimavictorPosted about 3 years ago

  @hardy333 thank you

  1