QR Code Component using Flexbox, SCSS and bem

Solution retrospective
- Since I'm not a beginner, I was able to complete the project quickly and efficiently. 🚀✨
- My familiarity with the technologies and my experience allowed me to work smoothly and confidently. 😊💻
- There were no significant difficulties during this project. 🌟
- Everything went according to plan, and I was able to implement all features without any major issues. 👍🛠️
- While there were no special difficulties, I would love to get advice from others on how I could improve. 🧩👨🏫
- Feedback on my approach and any suggestions for better practices would be greatly appreciated. 🌟💡
Please log in to post a comment
Log in with GitHubCommunity feedback
- @haquanq
Hello @DalaScript 🙋🙋🙋
Nice work, it looks perfect on the screenshot comparison!!
Here is something you can improve:
- When you use
article
, always give it a title/heading (oftenh2-h6
- each page must have oneh1
). In your case, these bold text is obviously the title/heading of thearticle
so you can useh1
instead ofp
. - You can just leave
h1
andp
without wrapping it indiv
and the layout still looks the same (avoid using unnecessarydiv
becausediv
has no semantic) .
Nice work, keep it up!! Happy coding 😁😁😁
Marked as helpful - When you use
- Account deleted
Great job, keep it up!
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