HTML CSS flex

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace<div class="container">with the main tag and <div class="attribution"> with the footer tag to fix the accessibility issues. click here for more on web-accessibility

  To center .container 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.

  Give .container a fixed max-width value in the mobile design max-width: 400px and a fixed width value width: 900px in the desktop design(media query). There is no need to give h1 a font-size and reduce the margin value of p p{margin-bottom: 30px;} .

  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

  Marked as helpful

  1