Order summary component main solution

Community feedback

• 

  Rio Cantre•9,650

  @RioCantre

  Posted over 3 years ago

  Hello there! Nice work with this one. Viewing your solution, I would suggest the following for you...

  • Add the background image in the body tag with the following properties...

  background-position: top center;
  background-size: cover;
  background-repeat: no-repeat;
  bacground-color: (add color value here);
  

  • In the .pay add color:white for the font and add background color for the hover state

  Hope this helps and Keep it up!

  02

  Marked as helpful
• 

  Anosha Ahmed•9,300

  @anoshaahmed

  Posted over 3 years ago

  hey good job on this challenge! wrap everything in your body in <main> OR use semantic tags! if you're using div right after body, then give them a role=""; however, it's a little frowned upon to use role right after body, so try to stick with semantic tags instead. you can read more about landmarks here

  here is a list i made of accessibility issues & best practices

  hope this helps :))

  01

  Marked as helpful