QR CODE COMPONENT MAIN

Hello there 👋. Good job on completing the challenge !

I have other suggestions about your code that might interest you.

• The Github repository is private or not available. :(

HTML 📄:

• Wrap the page's whole main content in the <main> tag.

• The <br> tag is often used to create line breaks, but it doesn't convey any semantic meaning. When a screen-reader reads the text, it will break the flow of reading at the line break tag, which can be confusing for users. More information here.

• You should not use inline-CSS because it is not a good practice. Instead, you should use an external stylesheet to style your page. By doing this, you will be able to have a better organization of your code and will be able to understand it better.

• Since this component involves scanning the QR code, the image is not a decoration, so it must have an alt attribute. The alt attribute should explain its purpose. e.g. QR code to frontendmentor.io

I hope you find it useful! 😄 Above all, the solution you submitted is great!

Happy coding!

Replace <div class="container"> with the main tag and add the alt attribute alt=" " to the img tag to fix the error and accessibility issue. click here for more on web-accessibility and semantic html

To center on the page, add min-height:100vh; display: flex; align-items: center: justify-content: center; or min-height:100vh; display: grid place-items: center to the body.

To center .container on the page using flexbox:
body{
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}

To center .container on the page using grid:
body{
min-height: 100vh;
display: grid;
place-items: center;
}

Hope am helpful.

Well done for completing this challenge. HAPPY CODING