order-summary-component built with HTML, Sass and Flexbox

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted almost 3 years ago

  Hi

  I recommend you

  • look up how and when to write alt text on images
  • use background img property for the big curve, get that out the html
  • add a little margin on the card or some padding on the body so your component doesn't hit screen edges on mobile
  • make annual plan a h2
  • choose more carefully when to use buttons and anchor tags. This might be fine, or it might not. All depends on what you would expect to happen when someone clicks change / cancel / or proceed?

  0

  ---

  

  Nuel•70

  @nuelobetoPosted almost 3 years ago

  @grace-snow thank you for taking the time to go through my solution and send me this well structured feedback. I'll work on the areas you pointed out. Thank you!

  0