Responsive QR code component using CSS flexbox and media queries

Community feedback

• 

  Francisco Carrillo•5,540

  @frank-itachi

  Posted over 2 years ago

  Hello khaykay samuel 👋. You did a good job!

  I have some suggestions about your code that might interest you.

  HTML 📄:

  • The heading order is important in the html structure so try to always start your headings and/or titles with an <h1> tag and then you can increase by one if you need to use more heading in your html code. So replce the <span class="title"> by the <h1> tag.
  • Put the the <div class="attribution"> section out of the <main> tag and wrap it inside the <footer> tag.
  • Get rid of the target="_blank" attriburte of the footer link.

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy <coding /> 😎!

  00