Design comparison
SolutionDesign
Solution retrospective
Give me feedback about my solution :)
Community feedback
- @ApplePieGiraffePosted about 4 years ago
Hey, Ananda Zulvansyah Putra! 👋
Nice job! Your solution looks pretty good and is responsive! 👍
I suggest,
- Setting a
max-width
on the main container or wrapper so that the content of the page doesn't look too stretched and the layout doesn't change too much on extra-large screens. - I think a little less padding to the left and right sides of the page in the desktop layout would give the content of the page some more room.
- Setting a
max-width
on the review boxes and testimonial cards might also be a good idea since they are quite wide when the layout first changes from desktop to mobile.
Keep coding (and happy coding, too)! 😁
1@azurasanPosted about 4 years ago@ApplePieGiraffe thank you for your suggestion, i'll try it later.
0 - Setting a
- @Greeshma2903Posted about 4 years ago
Good work with the design,
- you can add some more margin top and bottom margin to the
.container
, for a cleaner look - maybe you can add
overflow: auto
tocontainer
, to enable the scroll option for smaller screens. Happy coding! :)
1@azurasanPosted about 4 years ago@Greeshma2903 i already fix it, can u see it again for me?
0 - you can add some more margin top and bottom margin to the
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