Basic HTML and CSS page to show a QR code

Community feedback

• 

  Adriano•42,890

  @AdrianoEscarabote

  Posted 8 months ago

  Hello Morgan Walker, how are you? I was really pleased with your project, but I’d like to offer some advice that might help:

  Using Flexbox or Grid on the body to center elements ensures a more responsive and adaptive layout, fitting different screen sizes seamlessly. It avoids manual calculations and constant adjustments needed with margin, padding, or absolute positioning. These techniques provide more consistent alignment and simplify the code.

  flexbox:

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

  grid:

  body {
      display: grid;
      place-content: center;
      min-height: 100vh;
  }
  

  The rest is spot on.

  Hope it’s helpful to you. 👍

  00
• 

  jrybarski•210

  @jrybarski

  Posted 8 months ago

  Does the solution include semantic HTML? Is it accessible, and what improvements could be made? Does the layout look good on a range of screen sizes? Is the code well-structured, readable, and reusable? Does the solution differ considerably from the design?

  00