Design comparison
Solution retrospective
The feedback is welcome and very helpful
Community feedback
- @AgataLiberskaPosted over 3 years ago
Hi @AngelBLK!
The grid looks great, but I think it could use some width restriction or more padding/margin or something - on my desktop it stretches all across the screen and the cards have quite a lot of empty space. Not a massive issue, but it would look more polished. Also, the text breaks at weird points on the purple card, I can send you a screenshot on Slack if you want?
I also think it would be good to add an extra layout between the column and the full grid. The column looks good, I personally like that you've set max-width so the cards don't stretch too much. But there's a lot of empty space before the grid layout kicks in.
Oh and I just noticed that the margin disappears on smallest mobiles, I'd add that in, too.
Hope this helps :)
1
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