QR code component

Community feedback

• 

  geomydas•1,400

  @geomydasPosted 5 months ago

  only thing I would change is the div tag that has the class of .card into a <main> tag.

  Marked as helpful

  1
• 

  Fortune Iyoha•280

  @fortuneiyohaPosted 5 months ago

  Congratulations on completing the challenge! 🎉

  Improve Accessibility

  To enhance accessibility, it's important to structure your page content using semantic landmarks and tags. One suggestion is to use the <main> tag to wrap your primary content. Here's an example:

  <!-- index.html -- >
  <main class="card">
    ...
  </main>
  

  Improve Styles

  To ensure your content fits well on the page, it's a good idea to remove default margins. Using a CSS reset is a comprehensive solution, but for this case, removing the body margin and setting a minimum height will do the trick. Here's how you can update your CSS:

  /* index.css */
  body {
    ...
    min-height: 100vh;
    margin: 0;
    ...
  }
  

  Great job on your project! 😄 Keep up the excellent work and happy coding!

  Marked as helpful

  0