Responsive page, use Css Flex-Box

Community feedback

• 

  P

  Dilhan Boca•220

  @dboca93Posted 11 months ago

  Hello Ahmed,

  I hope your enjoying your coding experience. There is a particular issue with your code that if you correct, your code will look much closer to the design file. Your code on the body element is currently like below:

   body {
      background-color: hsl(217, 54%, 11%);
      display: flex;
      justify-content: center;
      align-items: center;
  }
  

  But if you change it to this:

   body {
      background-color: hsl(217, 54%, 11%);
      display: flex;
      justify-content: center;
      align-items: center;
      min-height: 100vh;
  }
  

  You should see an immediate difference. I didn't have time to review the rest of the code. However, I also noticed that you need to put a min-width (maybe around 320px) on the parent container. You can adjust the height/width of the image as needed. Also, if you have time learn about semantic html, as this is important in the future. Please if you find my comment helpful, mark it as helpful.

  Feel free to add me on twitter: @dboca93 Thanks !

  1