Design comparison
SolutionDesign
Solution retrospective
I did not face any difficulties in its work " Waiting for your comments "
Community feedback
- @ahmedafsaPosted 12 months ago
Hello @0xmaxx1, Well done!
Small suggestions to enhance the code further:
- Regarding the product image It is better to use the
<picture>
element with<source>
tags to make it responsive where based on your screen width, the appropriate image will be loaded instead of the two<img>
elements.
<picture> <source srcset="images/image-product-mobile.jpg" media="(max-width: 600px)" /> <source srcset="images/image-product-desktop.jpg" media="(min-width: 601px)" /> <img src="images/image-product-desktop.jpg"/> </picture>
- You can use
cursor: pointer
on the buttons andtransition: all 0.3s
on the elements with hover effect to apply nice hover effects
I hope these suggestions are helpful. Best wishes to you!
0 - Regarding the product image It is better to use the
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