Simple QR code component using CSS flexbox

Hello Kelly, You have done a good work! 😁

Some little tips to improve your code:

• use main tag to wrap the card and improve the Accessibility but not as a container of that one element
• also you can use article tag instead of a simple div to the container card for improve the Accessibility
• use min-height: 100vh to body instead of height, otherwise the content is cut off when the browser height is less than the content
• instead of using px use relative units of measurement like rem -> read here

Keep learning how to code with your amazing solutions to challenges.

Hope this help 😉 and Happy coding!

Hey @Hsin-tingCHI, some suggestions to improve you code:

• the height from the Body Element needs to be min-height.

• The Alt Tag description for the QR image needs to be improved upon. Its needs to tell screenreader users what it is and where it will take them to when they scan it.

• Change the width from .card__img to max-width: 100% to make it responsive.

• Change width from **.card ** to `max-width to make the card responsive.

• Delete all this from **.card ** its unnecessary and just bloating your code :

.card {
    display: flex;
    flex-direction: column;
    justify-content: center;
    align-items: center;
}

Happy Coding!