Order summary component challenge

Community feedback

• 

  Travolgi 🍕•31,280

  @denielden

  Posted over 3 years ago

  Hi Tejaswini, great work on this challenge! 😉

  Also remove all margin from .container class because with flex they are superfluous and use min-height: 100vh instead height to body

  Overall you did well! Hope this help 😁

  10

  Marked as helpful
• 

  Fluffy Kas•7,655

  @FluffyKas

  Posted over 3 years ago

  Heyo,

  Your solution looks great on all screens! The only thing I'd recommend is looking into writing more helpful alt texts. They're a substitute for people unable to see the actual images so they should be descriptive. If you can't come up with an appropriate description, it probably means that the image doesn't really add anything to the content and just there for decoration in which case it's better to leave the alt text empty - which, if you ask me, could be said of both images in this challenge. If you really would like to stick to having an alt text, you could add something like "Girl dancing and listening to music" to the top illustration ^^

  10

  Marked as helpful