Responsive image

Community feedback

• 

  Olaniyi Ezekiel•7,580

  @Ezekiel225Posted 10 months ago

  Hello there 👋 @Alex-Gamarano.

  Good job on completing the challenge !

  Your project looks really good!

  I have suggestions about your code that might interest you.

  Consider adding a min-height of 100vh to the body element so as to centralize your project.

  body { 
    min-height: 100vh;
    align-items: center;
    display: flex;
    justify-content: center;
  }
  
  

  I hope it helps!

  Other than that, great job!

  Happy coding.

  0

  ---

  

  Alex-Gamarano•350

  @Alex-GamaranoPosted 10 months ago

  @Ezekiel225 , thank you for the tip. I wsa practicing grid in this one, so after your suggestion I went and re-aligned items for a better distribution. The fact that I'm using the attribution as footer throw me off a bit.

  Cheers.

  0