QR page using HTML and CSS

Community feedback

• 

  3kori•80

  @3koriPosted 8 months ago

  Hi @MARCUS-IMPERATOR,

  Congratulations on completing this project. It looks amazing!

  I'd like to offer some suggestions to improve the solution:

  • Set these global styles to avoid spacing and layout issues caused by browser default styles.

  * {
      box-sizing: border-box;
      margin: 0;
      padding: 0;
  }
  

  • Great job on using flexbox to center the container. Add min-height: 100vh; to the body to help with the centering.
  • To avoid squishing the container when changing screen sizes, consider using a high percentage for the container's width.

  media screen and (width: 750px)
  .container {
      width: 95%;
  }
  

  • Finally setting padding for the body isn't necessary.

  I hope you find this helpful!

  Marked as helpful

  1

  ---

  

  MARCUS-IMPERATOR•90

  @MARCUS-IMPERATORPosted 8 months ago

  @3kori Thank you so much, I will incorporate these changes right now.

  0