Order summary component main solution

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 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!

  Marked as helpful

  0

  ---

  

  marvyn kiplagat•50

  @Marvyn21Posted almost 3 years ago

  @RioCantre Thank you for your feedback.

  1

  

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  @Marvyn21 No worries! I would appreciate it if you mark it as helpful. Cheers!

  Marked as helpful

  0
• 

  Anosha Ahmed•9,300

  @anoshaahmedPosted almost 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 :))

  Marked as helpful

  0

  ---

  

  marvyn kiplagat•50

  @Marvyn21Posted almost 3 years ago

  @anoshaahmed I appreciate the feedback.

  1