Responsive QR Code Section with Flex + Grid Layout

Solution retrospective
Hi, everyone It was really fun building this, I would appreciate it if you take a look at my work and tell me better ways to do it and make it closer to the design provided.
Thank you
Please log in to post a comment
Log in with GitHubCommunity feedback
- @correlucas
👾Hi @Ikziriv, congratulations on your solution!
I’ve few suggestions for you that you can consider adding to your code:
Reduce your code by removing unnecessary elements. The HTML structure is working but you can reduce at least 20% of your code by cleaning the unnecessary elements, you start cleaning it by removing some unnecessary
<div>
. For this solution you wrap everything inside a single block of content using<div>
or<main>
(better option for accessibility) and put inside the whole content<img>
/<h1>
and<p>
.<body> <main> <img src="./images/image-qr-code.png" alt="QR Code Frontend Mentor" > <h1>Improve your front-end skills by building projects</h1> <p>Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p> </main> </body>
✌️ I hope this helps you and happy coding!
Marked as helpful
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