social-links

Community feedback

• 

  Nikola•220

  @porumbachanovPosted 6 months ago

  There is no need for the margin: 18% auto; in the .container as it needlessly stretches the window. Instead, you can center the items using flexbox by adding the following lines in the "body":

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

  display: flex; Enables the flexbox layout.

  flex-direction: column; Specifies the flex container's main axis to column, which means that the child elements will be laid out vertically.

  align-items: center; Centers the items horizontally.

  justify-content: center; Centers the items vertically within the container.

  height: 100vh; Sets the height of the "body" element to occupy the full height of the viewport ("vh" stands for viewport height).

  Hope this helps :).

  Marked as helpful

  1

  ---

  

  Louise•130

  @atheenaoteyzaPosted 6 months ago

  @Vrondir wasn't really gonna use flex here but yea I should have even just for the container to center it thanks!

  1