Responsive product preview card using css grid, flex-box, media query
Design comparison
Community feedback
- @leoikeh99Posted over 2 years ago
Good solution, but i have a few corrections:
-
I think is just better to set the buttons width to 100%, since you have already set the width and padding for the container.
-
I think will also be better to set the width of the container to something like 90% when the screen reduces to a certain width, this will enable better responsiveness.
-
For the button, i don't think you should set the border-radius to 5%, rather is should be something like 8px.
-
Lastly you should reduce the line height of you <h2> a bit.
Marked as helpful0@Eslam-EOPosted over 2 years ago@leoikeh99 Thank you so much for your feedback, I made the changes and it really helped me a lot.
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