Flexbox, grid, mobile first

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace <h2> with <h1> and <h3> with <h2> to fix the accessibility issue.

  There is no need for min-height:100vh in the body. add min-height:100vh; to the main and remove margin-top 10vh from the main to center .container on the page. You forgot to change the font-color of p in .join community.

  Hope am helpful Happy coding

  Marked as helpful

  1

  ---

  

  John•210

  @gt78xPosted almost 2 years ago

  @Hassiai Thanks!

  0
• 

  Favour•2,160

  @Nadine-GreenPosted almost 2 years ago

  HEY JOHN!

  I noticed that you have an issue with centering, a quick fix for this would be the give the body a height of 100vh height:100vh then a display of grid display: grid and then using the code place-items: center

  HOPE I COULD BE OF HELP :)

  HAPPY CODING!

  Marked as helpful

  0