order-summary-component-main

Community feedback

• 

  Ritik Kumar•120

  @itswebcoderPosted almost 2 years ago

  This is accurate but in this code there will be issue in your height. I hope you find it useful! 😄 Above all, the solution you submitted is great! Replace <div class="main-body"> with the main tag, v <div class="heading"> with <h1>, <div class="par"> with <p>, <div class="text"> with <h2>, <div class="btn">with <button> and <div class="cancel"> with <a> add aria-hidden=" true" to the <a> to fix the accessibility. issue. click here for more on web-accessibility and semantic html .

  Marked as helpful

  0
• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace <div class="main-body"> with the main tag, v <div class="heading"> with <h1>, <div class="par"> with <p>, <div class="text"> with <h2>, <div class="btn">with <button> and <div class="cancel"> with <a> add aria-hidden=" true" to the <a> to fix the accessibility. issue. click here for more on web-accessibility and semantic html .

  To center .main-body page, add min-height:100vh; display: flex; align-items: center: justify-content: center; or min-height:100vh; display: grid place-items: center to the body.

  The background image of the body is not visible, to make visible add "." to the beginning of the url value. url(./images/pattern-background-desktop.svg) apply this to the music icon and add a bbackground-repeat: no-repeat and background-size: contain to the body.

  You forgot to give the proceed payement a box-shadow

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  0