QR code component using SASS

Greetings! 👋 Congratulations on completing the challenge.

I have some suggestions for improvements:

• Consider using min-height: 100vh in the body instead of height: 100vh. This is because height: 100vh may cause the component to be cut off on smaller screens.

Explanation:

• If we set height: 100vh on the body, then body will always be 100% viewport’s height regardless of the content. So if the content is more than 100vh, it would be cut off.
• If we replace this with min-height: 100vh and leave height as its default value, then the body’s height would minimally be at 100vh but can still grow beyond 100vh depending on the content.

I hope this helps and happy coding! 😊

Hello there 👋. Congratulations on successfully completing the challenge! 🎉

• I have other recommendations regarding your code that I believe will be of great interest to you.

QR iMAGE ALT TEXT 📸:

• The QR Code Component involves scanning the QR code, the image is not a decoration, so it must have an alt attribute which should explain the purpose of the image.

• The alt with qr code is not even explaining for what the QR image need to be used.

• So update the alt with meaningful text which explains like QR code to frontendmentor.io

• Example: <img src="/images/image-qr-code.png" alt="QR code to frontendmentor.io">

.

I hope you find this helpful 😄 Above all, the solution you submitted is great !

Happy coding!