Order Summary Component

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted about 1 month ago

  Good job, this looks quite close to the design, just a bit smaller which is likely due to padding differences on the larger screen view.

  One thing you definitely need to check is the html though. There are a few issues and questions for you:

  1. Alt text on images shouldn’t ever include the word “image” Because they are already on an element with an image role. You could say "illustration" perhaps but should be a little more descriptive of what this illustration looks like / is conveying if you think this is valuable content.
  2. Music icon is a decorative image so alt must be empty.
  3. Personally I would make annual plan a heading because it is acting as a heading for the content under it. The price then becomes a paragraph. (Remember you can use grid for the layout in that box if all elements are direct children or you can wrap those two bits of content in an extra element for layout with flex).
  4. Change plan must be an interactive element.
  5. It's not necessarily wrong but I'd like you to think carefully about the function of the interactive elements in this design. Forget about styling, just think about what they would do on click. Because that determines what elements you should be using in html. Should "change plan" be an anchor and should"proceed" be a button and what should "cancel" be? Think it through.

  Marked as helpful

  0