Summary project

Community feedback

• 

  Harshita Jain•380

  @Harshi786Posted over 2 years ago

  Hey! Congrats on completing the challenge.

  Your code looks good but few more tips to make it perfect:

  • For background image, change body: { background-size: contain; background-position: top; }

  • Add hover to cancel order button. .pagos__cancelar:hover { color: #1f2f56; }

  Hope this helps :)

  Marked as helpful

  1

  ---

  

  Yoshii•200

  @YoshiiCoding17Posted over 2 years ago

  @Harshi786 I'll keep in mind , thanks :3

  1

  

  Yoshii•200

  @YoshiiCoding17Posted over 2 years ago

  @Harshi786 I'll keep in mind , thanks :3

  0
• 

  Tesla_Ambassador•3,000

  @tesla-ambassadorPosted over 2 years ago

  Hey Yoshii this is pretty solid, it is really sharp! Just a few points ot note, incase you want to resolve your accessiblity issues, you might want to consider adding a <main> tag and you might also want to consider enclosing your <div> tags in a landmark like <section> but in your case, since you don't have a lot of content on your landing page, you could just enclose everything in a <main> tag to kill two birds with one stone.

  Marked as helpful

  1

  ---

  

  Yoshii•200

  @YoshiiCoding17Posted over 2 years ago

  @tesla-ambassador Thank you for the advices :3

  1