Design comparison
SolutionDesign
Solution retrospective
Greetings all,
Any feedback on anything is highly appreciated.
Is it correct that in grid layout, responsiveness of "auto-fit, minmax value" will not work because grid item is explicit place on different column?
Best regards, KT
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hello there, KT! π
Nice work on this challenge! π Yeah, you might have to be a little more explicit with CSS grid in this challenge since one card span two rowsβI'm no CSS grid expert though, so there could be a way, maybe... π§π
Besides that, I'd like to suggest,
- Adding a little more margin/padding between the sides of the grid and the edges of the page (so that there's a little more room between the two when the width of the screen decreases).
- Adding a max-width to the testimonial cards and centering them in the viewport in the mobile layout so that they aren't too wide when the layout first changes.
Keep coding (and happy coding, too)! π
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