All comments

• 

  Grigoreva Eva• 80

  @GrigoryevaEva

  Submitted over 1 year ago

  During the implementation of the project, I encountered a problem, namely the vertical centering of the element relative to the entire page.

  I'm not sure I chose a good solution. I wrapped the main element in a container, for which I set the following values:

     position: fixed;
     top: 0;
     bottom: 0;
     right: 0;
     left: 0;
     display: flex;
     justify-content: center;
     align-items: center;
  

  After that, the main element was centered vertically.

  

  Adaptive landing page using Grid and Flexbox

  2

  30

  

  Lidia• 280

  @lidimi

  Posted over 1 year ago

  Hi!

  You did a very good job on this project. I have some suggestions:

  To center this element both vertically and horizontally you can make use of the body tag. Just set the height of body to 100vh (100vh is 100% of the viewport height), and use flex.

  Also, remember to add alt attribute to your images. It's very important for screen readers.

  Marked as helpful

  1

  This comment has received 1 replyShow