All comments

Szymon Kubiak•10

@skubiak447Submitted almost 2 years ago

I have no idea why these two warnings are popping up for me:

Document should have one main landmark

2.All page content should be contained by landmarks

QR Code HTML & CSS

Patryk Sz•210

@dedkuPosted almost 2 years ago

1.Remove your style tag from head and move attribution class to your css file. More info: Here

2.Your body should look like this:

  <body>
   **</main>**
    <div class="card">
      <img alt="screen" src="./images/image-qr-code.png" />
      <h1 class="title">Improve your front-end skills by building projects.</h1>
      <p class="desc">
        Scan the QR code to visit Frontend Mentor and take your coding skills to
        the next level.
      </p>
    </div>
   **</main>**
  **<footer>**
    <div class="attribution">
      Challenge by
      <a href="https://www.frontendmentor.io?ref=challenge" target="_blank"
        >Frontend Mentor</a
      >. Coded by
      <a href="https://github.com/skubiak447" target="_blank">Szymon Kubiak</a>.
    </div>
  ** </footer>**
  </body>

move footer into the body tag, and wrap all your html code from beggining of body till footer in main tag. More info: Here

Cheers!

Marked as helpful