CSS Grid

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted about 2 years ago

  👾Hi Kritika, congrats on completing this challenge!

  Nice solution and nice code! I can see that you paid a lot of attention to your code/design. If you don’t mind I’ve some tips for you:

  Fix the alignment of the whole content using flex and min-height to manage the vertical alignment and make everything centered.First of all put min-height: 100vh to the body to make the body display 100% of the viewport height (this makes the container align to the height size that's now 100% of the screen height) size and display: flex e flex-direction: column to align the child element (the container) vertically using the body as reference.

  body {
      MIN-HEIGHT: 100VH;
      background-color: hsl(204, 43%, 93%);
      font-family: 'Karla', sans-serif;
      /* DISPLAY: FLEX; */
      display: flex;
      align-items: center;
      justify-content: center;
  }
  

  ✌️ I hope this helps you and happy coding!

  Marked as helpful

  1
• 

  რეზი კარანაძე•150

  @rezi-rezikoPosted about 2 years ago

  hello, great job keep going!

  I have some feedback for you my friend.

  you can use flexbox for moving main box in the center,

  make parent div display: flex; justify-content: center; and align-items: center; So your solution will move in center.

  Marked as helpful

  1