Design comparison
Solution retrospective
I feel like I overdid it using flex on every card section, ignoring simpler ways to position.
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hello, how are you? You did a good job on this challenge.
Don't blame yourself for using flex a lot, it's a good practice to align your layout. I really liked the way you coded the page, and the responsiveness of your project is also very good.
just a detail for the background image that is very large, you could have put it like this:
body { background-size: 100% 50vmin; background-position: top; }
To switch the image to the mobile version you could have used a media query.
The rest was very good.
hope it helps... don't forget to mark it as helpful. 👍
Marked as helpful1
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