QR code component

Community feedback

• 

  Francisco Carrillo•5,540

  @frank-itachi

  Posted over 2 years ago

  Hello there 👋. You did a good job!

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

  HTML 📄:

  • Wrap the page's whole main content in the <main> tag.
  • Wrap the <div class="attribution"> section inside the <footer> tag.
  • 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 replace the <p>Improve your front-end skills by building projects</p> by the <h1> tag.

  CSS 🎨:

  • You can use grid or flexbox to center the content no matter the viewport size. Using Grid:

  body {
  	min-height: 100vh;
          display: grid;
  	place-content : center;
  }
  

  FlexBox

  body {
  	min-height: 100vh;
          display: flex;
  	align-items: center;
  	justify-content: center;
  }
  

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

  Happy <coding /> 😎!

  00