QR code component

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace <div class="main"> with the main tag and <h3> with <h1> to fix the accessibility issues. for more on semantic html visit https://web.dev/learn/html/semantic-html/

  To center .main on the page using flexbox, add min-height:100vh; to the body.

  For more responsive width replace the width with max-width and increase the value to meet the desired width of the design. Give . main a padding value for all the sides instead of a height value. There is no need to give . image a width value , padding value, height value and a border-radius.

  There is no need to give img a height value . you can replace the width with a max-width.

  Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To

  Hope am helpful Happy coding

  0

  ---

  

  maixiang•10

  @maixiang-shPosted almost 2 years ago

  @Hassiai Thank you. I learned more about css from it.

  0