order-summary-component-main

Community feedback

• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as well…

  • Remove all the properties in the .container rule set and add it on the body rule set. Include background-size: cover;, background-position: top; and background-repeat: no-repeat;. Remove background-position: -800px;, height: 100vh; and width: 100%;
  • Add background-color: hsla(223, 41%, 95%, 1), padding: 1rem; and border-radius: 10px; in the .plan rule set
  • For HTML structures, use semantic tags accordingly. Refer it with this one Semantics
  • Include description with the alt in image tags. Like this line <img src="./images/icon-music.svg" alt="">

  Above all, Well done! Keep it going!

  0
• 

  Alazar Getachew•260

  @AlazarG19Posted almost 3 years ago

  i suggest you move your background a little higher u can do it by adjusting the background-position-y property

  0