
Design comparison
Community feedback
- P@rainbowsurfer137Posted 28 days ago
You’ve made great use of Tailwind’s utility classes. The md: breakpoints are effectively used for the mobile-first approach. Using picture for responsive images is a good choice. This ensures that the right image is served based on the screen size, improving load time and performance.
In the button class hover:hover:bg-[#243F32], there’s a duplication of the hover class. The hover pseudo-class doesn’t need to be repeated. Simply use:
<button class="add-to-cart-btn w-full h-3 bg-[#235B49] mt-5 min-h-10 p-2 rounded-lg flex items-center md:flex md:flex-row hover:bg-[#243F32]">
You’ve done a great job!
0@KarthikMgkPosted 27 days ago@rainbowsurfer137 Thanks for the detailed comment. I will definitely remove the duplication and reupload the solution. Thanks for the tip
0
Please log in to post a comment
Log in with GitHubJoin 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