Responsive flexbox product card

Community feedback

• 

  Adriano•38,290

  @AdrianoEscarabotePosted about 2 years ago

  Hi Cousnay, how are you?

  I really liked the result of your project, but I have some tips that I think you will enjoy:

  To center the content in the center of the screen, we can do this:

  body {
      display: flex;
      align-items: center;
      justify-content: center;
      flex-direction: column;
      min height: 100vh;
  }
  

  For the user who uses assistive technologies, such as screen readers, to indicate what the main content of the page is necessary to have the main tag, we can solve this problem by involving all the content with the main tag.

  The rest is great!

  I hope it helps... 👍

  Marked as helpful

  0

  ---

  

  Cousnay•320

  @Disney-BanjePosted about 2 years ago

  @AdrianoEscarabote thank you for the review and advice on my project. Highly appreciated. I have a question though why did you use:

  body {
  min-height: 100vh;
  }
  

  instead of

  body {
  height: 100vh;
  }
  

  I would like to know why.

  Thanks again.

  0