Advice Generator Application using HTML, CSS, Vanilla JS

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Hi Mark!

  Hey relax my friend, we can easily center it by giving the following properties to the body

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

  The idea is that it occupies minimally of height all the height of the screen for that reason 100vh and with flex already we centered it in the middle so much in horizontal and vertical.

  And to the main remove the margin and you will see that it is centered perfectly.

  Marked as helpful

  2

  ---

  

  Mark Escosura•70

  @mark-escosuraPosted over 2 years ago

  This did the trick! By removing the margin, It centered every element perfectly. Thank you.

  0
• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  By the way, if or if you must put an h1 inside your main since all pages must always have an h1. As soon as you make all these changes update the report and take a new screenshot that they give 5 free ones every month and with that it should be pretty good.

  Good luck, Mark!

  Marked as helpful

  1

  ---

  

  Mark Escosura•70

  @mark-escosuraPosted over 2 years ago

  @Sdann26 Oh I see, got it. From now on, for every page that I create, I will put an h1 because it is required. Thanks! I didn't know this.

  1