order-summary-component-main solution

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace <p id='title'> with <h1> to fix the accessibility issue.

  Give .bg a width of 100%. Replace the height value of .fee with padding top and bottom value. I will advice you use .bg as the background image of the body with a background-size of contain an background-repeat of no-repeat. this way the main can be centered on the page.

  To center the main on the 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.

  Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on CSS units, click here

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  For a responsive content, replace the width of the main with max-width.

  Marked as helpful

  0