flexbox

Community feedback

• 

  Naleeka•470

  @NaleekaPosted about 2 years ago

  Hi @NelJulgan 👋

  • Use <footer> tag to wrap up the elements in bottom of the body

  <div class="attribution">
      Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>. 
      Coded by <a href="https://www.frontendmentor.io/profile/NelJulgan">Neljulgan</a>.
    </div>
  

  • Use <main> instead of <header> since it's not the heading you are building

  <main>
    <div class="container">
  
        <div class="card">
          <div class="card-content">
            <img src="./images/image-qr-code.png" alt=" qr-code image" class="qrcode"  aria-label="qr-code">
            <h3 class="card-title">Improve your front-end skills by building projects</h3>
            <p class="card-text"> Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p>
          </div>
  </main>
  

  • instead of using alt=" qr-code image" describe where it will be headed, like this 👉 alt="qr-code to FrontendMentor"

  Hope this was helpful

  Happy Coding Buddy 👩‍🚀🖤

  0

  ---

  

  Nel Julgan•130

  @NelJulganPosted about 2 years ago

  hi @Naleeka thanks a lot!

  1