QR-Code-Component

Hello there 👋. Good job on completing the challenge !

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

• Without the Figma design, estimating sizes can be challenging. I recommend using a browser extension like "Pixel Perfect" to help you more accurately estimate sizes.

Alt text 📷:

• To make the alt attribute as useful and effective as possible, avoid using words such as "image", "photo", or "picture" as they are redundant because the image tag already conveys that information. Instead, try to make the description as human-readable and understandable as possible.

  The alt attribute should explain the purpose of the image, for example, in the case of a QR code, a description like "qr code to frontendmentor.io" would be more appropriate.

  If you want to learn more about the alt attribute, you can read this article. 📘.

CSS 🎨:

• You don't need to add font-family: "Outfit", sans-serif; to each <p>, <h*> separately. Instead, add it to body.

• To center the component in the page, you should use 100vh instead of 100% and remove the margin:

body {
    background-color: hsl(212, 45%, 89%);
    display: flex;
    align-items: center;
    justify-content: center;
    min-height: 100vh;
    margin: 0;
}

.box {
    /* [...] */
    /* margin-left: auto; */
    /* margin-right: auto; */
    /* margin-top: 80px; */
}

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

Happy coding!

Hello ggitaun👋 congratulations on completing this challenge 🎉

I have some tips that you might find useful. Try to make your html more Semantic by wrapping the main page content inside a <main> tag . Replace <div class="box"> With <main class="box">. This changes nothing visually but Using it makes all the difference. Using semantic markup improves

SEO

And user experience for people using assistive technology such as screen readers. In order to make your HTML code more semantic, wrap the main title with <h1> instead of just h3 Here's a quick guide on how to use them:

The <h1> to <h6> tags are used to define HTML headings. <h1> defines the most important heading. <h6> defines the least important heading. Only use one <h1> per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with <h1>, then use <h2>, and so on. You can then style them in you css. Unlike popular belief.

Unlike what most people think, it's not just about the size and weight of the text

I hope this helps.

Your solution is nice👏

Happy coding 🙃