Order-Summary-Component

Community feedback

• 

  darryncodes•6,370

  @darryncodesPosted almost 3 years ago

  Hi Dracula,

  Overall it's a good solution, all the elements are there - well done!

  I would just be careful setting fixed heights and widths. It doesn't really play well with responsive design. The content should set the height and width controlled dynamically with max-width. This is a pretty helpful RWD course.

  You could also clear down you accessibility/html report if you add an alt-text, a <h1> and some semantic html. Heading structure and semantics are really important fundamental concepts.

  All the best!

  0
• 

  Nguyễn Duy Hùng•450

  @banhmisg9509Posted almost 3 years ago

  I think it'll look better if you add a transition property when hovering the button, the "change" and "cancel" labels. The padding of the card isn't same as the design, It should be bigger. The space between the music icon and the price tag (annual plan) also is quite far. I hope this comment can help you.

  0