Order-summary-desktop first

Community feedback

• 

  P

  Hamza H.•460

  @hhamza1Posted over 3 years ago

  First of all, good job on the delivery.

  Although you started with a desktop approach, you could still have a couple of things to set and you won't have to do much on the mobile view. Setting the display flex on the body would have made your work much easier, also you could have set a max-width to the card, that would have helped also with the responsiveness, box shadow should be fixed, bg-color and pattern could be also fixed.

  You might check my solution for more details : https://www.frontendmentor.io/solutions/order-summary-card-component-with-flex-MHX37axfl

  My approach was mobile-first view. It's much easier to expand to other screen sizes afterwards, not much CSS to write.

  Keep up the good work

  Marked as helpful

  1
• 

  Gautam•105

  @GautamjiPosted over 3 years ago

  This is how I did it. https://github.com/Gautamji/frontendmentor/tree/master/order-summary-component-main

  and my approach while making it. I live streamed it :D https://youtu.be/24ImgsJfLAY?t=8269

  Marked as helpful

  0