QR code Compon - made with Flexbox

Hi @IvanoCro,

Congratulations on your solution, it looks very close to the design specifications. Your code is clean and easy to read.

A couple of friendly suggestions:

• Instead of using a div for your card you could use article to make it more semantic.

• I suggest using min-height: 100vh instead of height: 100vh on your body. So you don't run into overflow issues if the card gets bigger. It's not a big problem in this example, just a good practice.

• I suggest keeping the height of the card as auto and not setting a px value. If you add more text to the description you run into overflow issues.

• You could try to use CSS variables to make your style guide values more reusable.

I hope you find my feedback helpful and I'll be happy to answer any other questions you have.

Hi @IvanoCro, Your code and design look great. The only minor detail would be to put the same padding that you have on the sides on the top. It's a minor detail but it would make the design look much cleaner visually. Other than that it looks great, good job!