Design comparison
Solution retrospective
please give your suggestions.
Community feedback
- @GenildocsPosted over 1 year ago
Hi there! Congratulations on completing the challenge!
As a suggestion, I believe you could have used display grid to define the entire layout of the page and use flex-box only on one-dimensional elements. For example, by applying grid to the "main" block, you could align the elements more efficiently.
For the main image, I recommend not defining a fixed height and instead letting the element take up 100% of its parent element or using the max-height property for better responsiveness to different screen sizes.
Lastly, I suggest using grid in the "bottom" section and flex-box on child elements for a better outcome.
I hope these suggestions are helpful in improving your work!
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