qr-code-component-main

Community feedback

• Account deleted

  hi @gregoryamaefule! good job on this challenge! it look great!

  I see some accessibility issues, I'm going to make you some suggestions:

  • First of all, the main content of your page should be enclosed in a <main> tag, I would make this change:

  <div class="container"> <-- delete

  <main class="container" --> add >

  • then, all pages must include a level-one heading, in this case, I would change the <h2> tag of the title for a <h1>:

  <h2>Improve your front-end skills by building projects</h2> <-- delete

  <h1>Improve your front-end skills by building projects</h1> --> add >

  I hope this feedback has been useful to you! keep coding :)

  Marked as helpful

  0

  ---

  

  gregory amaefule•220

  @gregoryamaefulePosted over 2 years ago

  @nachogonzalia thanks

  1