Order summary component main

Community feedback

• 

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  Nice work Benjamin!

  Your card container seems very close to the design.

  Few suggestions though:

  • I've noticed you have used fixed heights in many places. Try to use only min-height when necessary, but most of the time you don't need any height. Height should be decided by the content of the container, if needed simply tweak it with paddings and margins of the contents. Fixed heights are known to cause various issues, resulting in overflowing content when data or font-sizes change.

  • Add a cursor: pointer; to the button and link elements, it is an important indicator of interactivity, combined with a color/bg-color change on hover. It improves UX.

  Hope you find this feedback helpful!

  0