QR-Code-component-main

Community feedback

• 

  Dolapo•890

  @nofeesahdeePosted almost 3 years ago

  Great job Kevin, For the accessibility, instead of using div, try use semantic html5 such as main

  Marked as helpful

  1

  ---

  

  Kevin•160

  @Kevin-KigothoPosted almost 3 years ago

  @nofeesahdee Thank you for your feedback. Let me apply that when I redo it.

  0
• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted almost 3 years ago

  Hey,

  Good job on this one, looks pretty close! Perhaps it would be a better idea to add the image to the HTML instead of using it as a background since this QR code pic happens to be the main content of this whole component :)

  Marked as helpful

  0

  ---

  

  Kevin•160

  @Kevin-KigothoPosted almost 3 years ago

  @FluffyKas Thank you for the appreciation. Let me try that.

  0