Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

iterm centering using flexbox

salimβ€’ 10

@errondo

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

@gabrielmorandi

Posted

Hello @errondo! πŸ‘‹, welcome to the Frontend Mentor Community!

Congratulations on your QR Code component project! πŸŽ‰ It looks clean and well-organized. However, I would like to suggest some improvements that can further enhance your code:

  • HTML Semantics: Using semantic elements in your code can help browsers and assistive technologies better understand the structure of your document. In your case, you can use the <header>, <main>, and <footer> tags to clearly separate the different sections of your component.
<header>
</header>
<main>
</main>
<footer>
</footer>
  • Image size: Currently, you have set the image width to 300px, which might cause responsiveness issues on smaller devices. I suggest using max-width and width: 100% to allow the image to resize according to the device's size:
.item img {
  max-width: 300px;
  width: 100%;
  border-radius: 20px;
}
  • Alternative text: It's a good practice to include a more descriptive alternative text for your images. In the case of the QR Code, you can improve the alt attribute as follows:
<img src="./images/image-qr-code.png" alt="QR Code for Frontend Mentor website">

I hope these suggestions are helpful! Feel free to ask questions if you need more information. Good luck and keep up the great work! πŸ˜„

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord