Responsive Order Summary Page

Community feedback

• 

  Santiago•140

  @santiagosgPosted almost 3 years ago

  Hi Anna

  I think you use too much the div for elements that don't need it, like in the plan (the image and the plan-change).

  You are using a button to "proceed payment" and "cancel order", that's good and its ok, but remember to add a "cursor: pointer;"

  Try to organize your documents, like all the images in the images folder, and maybe your style.css under a css folder too.

  You are doing great, keep moving.

  P.D. I liked your way to show the screenshot in GitHub

  Marked as helpful

  0

  ---

  

  Anna Karina Angelim•20

  @akarinaaPosted almost 3 years ago

  @santiagosg thank you very much for your insight, Santiago! It's really helpful. I'm learning ways to make my code/project cleaner, so your advice is very appreciated.

  I chose a mockup to showcase the design because I'm really like how the mobile version turned out, so I wanted to show it along with the desktop version. I might do this for my next projects as well.

  0