Design comparison
Solution retrospective
I'm already familiarized with flexbox so now I'm trying to practice some CSS Grid and I'd really appreciate some tips or just something that I could've done better. I think that I've got it almost pixel-perfect on the desktop so I'm really proud. Also, I used SASS because it's really fun :).
Community feedback
- @ApplePieGiraffePosted almost 4 years ago
Hey, Guilherme Magno! 👋
Great work on this challenge! 👏 Your solution looks very good and is responsive! Glad you enjoyed making it! 😀
I only suggest adding a little more space around the sides of the grid so that it is not so close to the edges of the screen when the width of the screen decreases. Also, I think either adding a separate tablet layout with CSS grid or simply giving the testimonial cards a
max-width
in the mobile layout (so that they aren't too wide when the layout first changes) would be a nice idea! 😉Keep coding (and happy coding, too)! 😁
1@devMagnoPosted almost 4 years agoThanks for your feedback, I've added a tablet layout and it looks pretty cool! Also did some minor changes on padding and margins and the overall layout looks great! Thanks! Happy coding bro! 😉👊
0@ApplePieGiraffePosted almost 4 years ago@devMagno
Just took another look, and I like those changes! 👏 It looks great, as you said! 🙌 Keep it up! 👍
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