order-summary-component

Community feedback

• 

  Favour•2,160

  @Nadine-GreenPosted almost 2 years ago

  HEY LOGESH!

  Your solution is quite nice, however, I couldn't help but notice that none of the active states were included, if you look inside the design folder, you will see an image named, 'active-states', this image will show you which elements should have an effect.

  To get rid of your accessibility warning, instead of using a div for the .attribution, you should consider using a more semantic element like footer.

  HOPE I COULD BE OF HELP :)

  HAPPY CODING!

  Marked as helpful

  0