Frontend Mentor - QR code component with pure HTML5 and CSS3

Community feedback

• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  Hay ! Good Job Srikar Gunnam

  These below mentioned tricks will help you remove any Accessibility Issues

  -> Add Main tag after body like it should be your container. For 1st heading or h1 tag, use header tag and then inside the header put your h1 or h2 etc . But use header tag only once in main heading element.

  Keep up the good work!

  Marked as helpful

  0

  ---

  

  Srikar Gunnam•150

  @srikargunnamPosted almost 3 years ago

  Hi Naveen Gumaste, should i include both Header and Footer inside the Main tag?

  Thanks in advance 🙏

  1

  

  Naveen Gumaste•10,480

  @NaveenGumastePosted almost 3 years ago

  @srikargunnam you can & you can put it outside too..

  Marked as helpful

  0
• Account deleted

  Hi there 👋 Congratulate on finishing your project 🎉. You did a great job 🔨. Your readme is great. Everything is included

  I give some suggestions that I hope help you take your project to the next level 📈. About your design there is additional space at bottom of the card, maybe you should fix it also

  Happy coding ☕

  Marked as helpful

  0