Order Summary Component

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi, Ana! 👋

  It's been a while since you submitted the last solution, it's great to see you submit another solution! Good effort on this one! 👍

  Some suggestions.

  • In this case, all images are decorative images. So, leave all the alt empty so that the screen reader users won't have to listen to the non-related content.
  • Always specify the type of the button. In this case, set the type of them as type="button". It's going to prevent the button from behaving unexpectedly (like submitting).
  • Next time, try to write your code with consistent style (e.g. the indentation, quotes, whitespace, etc). If you write your code with consistent style, it will make it easier to read for everyone (including your future self). You could install a code formatter extension in your code editor to help you with this.

  That's it! I hope this information is useful! 😁

  Marked as helpful

  1

  ---

  

  Ana Vela•20

  @ana-velaPosted over 2 years ago

  @vanzasetia Thanks so much for remembering me, Vanza! This feedback helps a lot and I've made the changes you suggested. And I'm already planning my next challenge which will be one level up from these first two I've completed. :) Best wishes to you!

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @ana-vela I took a look at your code and good job with the updates!

  That's a spirit, Ana! Looking forward to your next solution! Good luck! 👍

  1
• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • add to the body background-size:contain;
  • add tranistion:1s; to the button so that when you hover on the button the background changes smoothly

  i hope this is helpful and goodluck!

  1

  ---

  

  Ana Vela•20

  @ana-velaPosted over 2 years ago

  @Old1337 Thanks so much for your feedback! I implemented these changes.

  1

  Account deleted

  @ana-vela good job!!

  1