Frontend Mentor | QR Code component | by TmT

Community feedback

• 

  John Idenyi•250

  @JohnIdenyiPosted over 2 years ago

  Nice work! Though you have some accessibility issues.

  • You don't have an h1 element in your document. Change the h2 to an h1 element. Every page must have an h1 element because screen readers make use of it to identify the main content to visually-impaired persons. It also helps in better optimization by search engines.

  • Your html root element is meant to have a lang attribute set to a value, for example lang="en" indicates the page is in english.

  I hope this was helpful.

  Marked as helpful

  1

  ---

  

  TmT•40

  @TimPavaPosted over 2 years ago

  @JohnIdenyi Thank you very much for taking the time to give me feedback ❤️

  Indeed, I have some accessibility issues ...

  Now, thanks to you, I understand them better and will rectify them in the future during new challenges 😉

  0

  

  John Idenyi•250

  @JohnIdenyiPosted over 2 years ago

  @TimPava Glad I was able to help!

  0