QR Code component solution (React/TypeScript/Tailwind)

Solution retrospective
Is there anything I can do better?
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@Milleus
Tailwind 😍 nicely done!
I have a couple of minor suggestions:
- The
flex flex-col
can be removed as the elements would naturally stack.
<div class="flex flex-col bg-white p-5 border rounded-3xl w-[24rem]"> // can be updated to <div class="bg-white p-5 border rounded-3xl w-[24rem]">
- Tailwind has a space utility that acts like flex/grid's
gap
, but without having to add the flex/grid property.
<div class="flex flex-col text-center gap-6 p-7"> // can be updated to <div class="text-center space-y-6 p-7">
Of course if the
flex flex-col
was added with intention, that's perfectly fine as well 😊Happy coding o/
Marked as helpful - The
- @PaliTriesToDesign
Congrats! I like your solution.
I would add just one more thing:
box-shadow
to the card. Nothing important.c:
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