QR Code Challenge

Community feedback

• 

  Yaser47•260

  @Yaser47Posted over 2 years ago

  Good work pro

  • if you want to central the main tag horizontal and vertical, now you don't need to margin-top in .square body { display: flex; justify-content: center; align-items: center; min-height: 100vh; }
  • And you are using the the wrong background-color.
  • And you can give the all element padding/ margin= zero, and if you want to modified any of them you can give specific value of top or bottom or right or left
    • { padding: 0; margin: 0; }

  0
• 

  Indra Adi Kusuma•910

  @indraAKPosted over 2 years ago

  to centering the first content, set height body and html to 100%, and then for class main add some properties min-height: 100%; display: grid; place-content: center; and also remove the margin-top for class message.

  0