QR Code

Community feedback

• 

  Adriano•42,890

  @AdrianoEscarabote

  Posted over 2 years ago

  Hi Marek Brzezinski, how are you? Welcome to the front-end mentor community! I really liked the result of your project, but I have some tips that I think you will enjoy:

  Avoid using px, If your web content font sizes are set to absolute units, such as pixels, the user will not be able to re-size the text or control the font size based on their needs. if you want to keep using px for development and then format the whole code to rem, you can use this vscode extension: px to rem

  As this is the main image of the page it cannot have an empty alt, images must have alt text unless it is a decorative image!

  <img class="qrCode" src="images/qr-code.svg" alt="" />

  The rest is great!

  I hope it helps... 👍

  11

  Marked as helpful
• 

  Amobi Jonathan Chukwudi•350

  @Jonathanthedeveloper

  Posted over 2 years ago

  Hey Marek, I noticed that you used a lot of divs . so you can refactor your code to

  
          <div class="cardContent">
            <h1>Improve your front-end skills by building projects</h1>
              <p>
                Scan the QR code to visit Frontend Mentor and take your coding
                skills to the next level
              </p>
          </div>
  
  

  instead of

  
          <div class="cardContent">
            <h1>Improve your front-end skills by building projects</h1>
            <div class="cardContent">
              <p>
                Scan the QR code to visit Frontend Mentor and take your coding
                skills to the next level
              </p>
            </div>
          </div>
  

  You

  11

  Marked as helpful