Simple QR Code reader solution

Community feedback

• 

  romila•3,570

  @romila2003Posted about 2 years ago

  Hi Caio,

  Congratulations 🎉 for completing this challenge, the card looks. There are some issues I want to address:

  1. It is best practice to wrap the main content within the main tag which would ensure that your content is wrapped within the correct landmarks e.g. <main class="container"></main>
  2. When using the ol or ul tags, you must only nest the li tags within it. In your ul tag, you have a div which is causing accessibility issues.

  Overall, great attempt and wish you the best for your future projects so keep coding 👍.

  Marked as helpful

  0