Responsive static QR code landing page

Community feedback

• 

  Olaniyi Ezekiel•7,580

  @Ezekiel225

  Posted over 1 year ago

  Hello there 👋.

  Good job on completing the challenge !

  Your project looks really good!

  I have suggestions about your code that might interest you.

  Consider changing the background-color of the body element to hsl(212, 45%, 89%)

  body {
    background-color: hsl(212, 45%, 89%);
  }
  

  I hope it helps!

  Other than that, great job!

  Happy coding.

  00

  Marked as helpful
• 

  Joyal James•265

  @silkcoder

  Posted over 1 year ago

  Your solution looks good

  I have a suggestion about the max-width property you given to body tag.

  If your solution opened in a screen with width greater than 1440px, the content not get centered. So you can remove the max-width property of body tag

  body {
     //remove
     max-width: 1440px;
  }
  

  Once you remove this property, your content remains on cener for all sized devices.

  Another point

  To center the container its wise to use Flexbox properties, read more about it on the page https://www.geeksforgeeks.org/introduction-to-css-flexbox/

  00

  Marked as helpful