Simple Qr code container

Community feedback

• 

  Olaniyi Ezekiel•7,580

  @Ezekiel225Posted 10 months ago

  Hello there 👋 @danielnunes13.

  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.

  Marked as helpful

  0

  ---

  

  Daniel Nunes•50

  @danielnunes13Posted 10 months ago

  @Ezekiel225 Hello, thx for the tips, it really helps.

  0
• 

  Craig•190

  @cchiversPosted 10 months ago

  Awesome job, I would say with the code: Something this small does not need to be broken down into different files. You had 2 css files for the project that are involved in this project/challenge. The reset css file is ok and I think you should slowly get away from using to help teach about how to manipulate different CSS and how they interact together.

  Marked as helpful

  0

  ---

  

  Daniel Nunes•50

  @danielnunes13Posted 10 months ago

  @cchivers Hello, yea i was a bit lost at the start, was trying to make it like some project i had participate, ty for the tip.

  0