Design comparison
SolutionDesign
Community feedback
- @DolbyTheSheepPosted 4 months ago
Well done, your code is really good! Nice use of CSS variables.
I would suggest the use of the
picture
tag for the images:<picture> <source media="(min-width:768px)" srcset="./images/image-product-desktop.jpg"> <img src="./images/image-product-mobile.jpg" alt="Image product" /> </picture>
And the use of
button
, which I think has the correct semantic, instead of adiv
with thebtn
class.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