Order Summary Component Main

Community feedback

• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey good job completing this challenge, keep up the good work.

  Your solution looks great however I think your card needs some margin from the top using margin-top property.

  In terms of your accessibility issues, simply wrap all your content between main tags

  I hope this helps

  Cheers Happy coding 👍

  Marked as helpful

  1

  ---

  

  Leonel Montiel•40

  @leonelmontielPosted over 2 years ago

  @TheCoderGuru Thank you very much for the good vibes! Regarding the top edges of the image, I have solved it with what was proposed by the colleague above. And I already corrected the <main> tag, thanks for checking that, I had forgotten to use it!

  Happy coding to you too :D

  0
• 

  DanielK•440

  @DanK1368Posted over 2 years ago

  Hi Leonel,

  You have to apply the top borders to your illustration image. border-radius: 10px 10px 0px 0px; Then it should work.

  Also to get rid of the Accessibility issues you have to wrap your content in a <main> tag. The order summary text should be an <h1> Tag.

  Hope this helps a little.

  Happy Coding

  Marked as helpful

  1

  ---

  

  Leonel Montiel•40

  @leonelmontielPosted over 2 years ago

  @DanK1368 Your solution about the edges was so simple haha, I have already applied it, thank you very much! And regarding the <main> tag I've already solved that too, honestly it slipped my mind.

  0