QR Code Component

Community feedback

• 

  Dipesh•130

  @ddkogitPosted 11 months ago

  use flexbox for this kind of layout, it will be much easier.

  1
• 

  Kimo Spark•2,190

  @kimodev1990Posted 11 months ago

  Great work in completing the challenge, Just a few feedback :

  • To center your container in the middle of your website, you could add in body :

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

  then the container will be centered, no need to use margin: 15px auto ;

  • Wrap your design in main tag.
  • You could use clamp ( ) method in your coding for font-size, width, margins, padding, etc., So the designed sizes will change according to the viewport dimensions having a responsive design and will be suitable for any device layout.

  Hope you find this Helpful.

  Other than that, Really Nice work & keep Going on

  1