qr-code challenge attempt

Community feedback

• 

  Hassia Issah•50,390

  @Hassiai

  Posted over 2 years ago

  To center .card on the page, add min-height:100vh; display: flex; align-items: center: justify-content: center; or min-height:100vh; display: grid place-items: center to the body.

  To center .card on the page using flexbox:
  body{
  min-height: 100vh;
  display: flex;
  align-items: center;
  justify-content: center;
  }
  

  To center .card on the page using grid:
  body{
  min-height: 100vh;
  display: grid;
  place-items: center;
  }
  

  Give .card a padding value for all the sides and the img a max-width of 100%, there is no need to give .card a margin value.

  Give h1 the same font-size as p.

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  00

  Marked as helpful
• 

  Imran Khan•370

  @astr0n0mer

  Posted over 2 years ago

  Hey @n-uzumaki, you're off to a great start

  HTML:

  • You can replace the <h5> with an <h1> for better accessibility
  • Do the styling in CSS

  CSS:

  • You can explore CSS naming conventions, namely BEM. This will make your code more readable for yourself and for others. Don't get me wrong, your code is readable even now, but BEM or any other convention for that matter, will definitely be helpful

  Happy coding. Good luck 😁😀

  01

  Marked as helpful