QR code component using HTML and CSS with Flexbox

Hi Carlos, great job! Congratulations on completing the challenge.

I had a look at your solution and I have a few suggestions for you:

• add main tag and wrap the card for Accessibility
• try to remove all margin from container class and use flexbox to the body for center the card. Read here -> flex guide
• also set heigth of body to 100vh because Flexbox aligns to the size of the parent container.

Also avoid assigning specific properties to the div element, rather create a class.

Overall you did well! Hope this help and happy coding!

1. Always wrap your main content inside the <main></main> tag like this

<main><img src="image-qr-code.png" alt="QR code" width="250" height="250"> <p class="p1"><b>Improve your front-end <br> skills by building projects</b></p> <p class="p2">Scan the QR code to visit Frontend <br> Mentor and take your coding skills to <br> the next level</p> </main>

2.don't use inline styles as you have done in the <img> element 3. use relative widths and max-width property. 4. set the font family inside the body element .