QR-Component

Community feedback

• 

  EthanAmato•130

  @EthanAmatoPosted almost 2 years ago

  Hey there! First of all I want to say fantastic job! It looks absolutely great! If you want to address some of the warnings that the accessibility report gave you, check out the following:

  "Document should have one main landmark" + "All page content should be contained by landmarks" Essentially, it's telling you to make use of semantic HTML (documentation here) to make the site easier to read for screen readers. Essentially, replace the <div class="container"> with a <main/> tag and you should be all set!

  2 "Page should contain a level-one heading"

  I believe this error is coming from a lack of an H1 tag in your code. Try replacing the H3 tag with an H1 (and swap the associated styling)

  Overall, fantastic job and I hope you enjoy your journey making websites :)

  0