QR code component with CSS and HTML

Community feedback

• 

  romila•3,570

  @romila2003Posted about 2 years ago

  Hi Casey,

  Welcome to the frontend mentor community and congratulations 🎉 for completing this challenge, it was a great attempt. I found 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. You should also wrap your footer within the footer tag e.g. <footer class="attribution></footer>

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

  0

  ---

  

  Casey•80

  @casey-botPosted about 2 years ago

  @romila2003 Thank you so much for the feedback! I'll be sure to double check those in other projects.

  1