QR Code - [HTML, CSS, SASS, Flexbox, CSS Grid]

Solution retrospective
📕 Any feedback is welcome and appreciated!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @dimar-hanung
Hi there 👋, congrats, Congratulations on completing the challenge, i have some feedback might useful for you.
That i like:
- first i like your slicing result is very similar, just little difference in shadow and font.
- i like the way you separate sections into a structured folder
- the responsive is pretty good, even until galaxy fold 🙌
My feedback:
- my opinion the comment in your code is not needed, example
<!-- CARD QR IMAGE ->
, because your classname that you created already represents what it is for, exampleclass="card__qr-link"
, by removing the comment it will make it clearer
anyways good solution!, i hope my feedback can be useful, thanks 🙌
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