Design comparison
SolutionDesign
Community feedback
- @SurajHadagePosted over 2 years ago
Whenever making single designs like this make inside <main> tag. Or use role attribute. Give overflow: hidden; to your container and border-radius, So you do not have to give it to both child elements separately. Use <picture> tag for image's responsiveness, It will be much easier. don't give margin to individual text, instead use padding to .product-text. To guess accurate padding use paint app ruler and status bar view. And also I suggest to not upload unnecessary files to github to save server space. This is my suggestion for you.
Marked as helpful0
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