Frontend Mentor - Product preview card component solution

Solution retrospective
Review my code and suggest improvements to apply it to a real-world project. I am a newbie in this field. Please help me out, and criticism is also welcome😊
Please log in to post a comment
Log in with GitHubCommunity feedback
- @Aimal-125
In your CSS code, just give
padding: 1rem;
to <body> element by using media query widthmax-height: 500px;
for small heighted screens as mine (360 X 541 pixels). And give .attribution class padding of 0.5rem so that it nicely comes down a little bit from the card component. Otherwise your card is looking great. Thumbs up! ::)Marked as helpful - @fibonacci001
hi! Pandey, Your code is functional and interactive. Good job on using the data-icon attribute and the ::before pseudo-element to create a custom button with an icon.👍 There are some minor areas where you can improve the code, here is an example:
.product__price { font-size: 2rem; } .flex-group { gap: 1rem; }
Overall, your code is well-written and meets the design specifications. I hope you find my feedback helpful.
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