Design comparison
SolutionDesign
Solution retrospective
Need review of this project to improve
Community feedback
- @a1excpunkPosted about 1 year ago
Hello there, congrats on completing this challenge,
- one thing I would suggest is to use grid
{display: grid;}
instead of flex. flex can be very useful and easy to use but in some cases, it makes more "trouble" than good. - the reason I'm suggesting a grid is that, on screen size change cards squish and space between cards becomes uneven. with the grid it gonna be smoother and easier. you'll need only one parent container and 5 cards that's it, no additional containers and headache on how to position them correctly.
- A Complete Guide to CSS Grid This is a very good source for understanding grid, for any level.
Keep going, keep learning and of course keep coding.
0@aliseidu8855Posted about 1 year ago@a1excpunk Thank you very much for the feedback i will learn that
1 - one thing I would suggest is to use grid
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