Submitted over 4 years ago
Creating a responsive card layout using css grid
@Peterklink
Design comparison
SolutionDesign
Solution retrospective
First time using the Grid display. Please let me know if this was the right choice of layout and if my code is good practice of coding.
Community feedback
- @rfilenkoPosted over 4 years ago
Hey, a few points for you:
- add transition to card to less staggering effect on hover or just remove scaling (it's not a link, so why there is interactive effect?);
- add just gap between cards;
- there are overflow issues on different viewport width;
- check how to reset browsers default styles.
Keep coding, Roman
1@PeterklinkPosted over 4 years agoHey, thank you for your feedback! I solved the first 3 problems you mentioned, but what do you mean with resetting browser default styles? Which styles exactly are wrong?
Anyways,
have a good one!
0@rfilenkoPosted over 4 years ago@Peterklink properties like margin, padding, box-sizing and others, allowing to have consistent styles across different browsers 😉
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