Design comparison
Solution retrospective
This is the first time I attempt to do a responsive page, I'm still kinda lost so I welcome any advice/comment/critique. Thanks
Community feedback
- @mathcrlnPosted over 3 years ago
Hello Daniel, well done!
I don't have much to say, your design is very well implemented and especially smooth. Nicely done for the first time! Additionally, I checked out your code and it's very easy and agreeable to read with the spacing and the comments. It seems clean and simple, yet effective. Congrats 👏🏾
When it comes to feedbacks, around 1125 pixels, things can seem a bit constrained and can seem to "jump" at times. I see that you added varying padding on the .card__content class. Maybe instead you could a div for the information and rather play with the max-width to get the effect you want?
Between 700 and 1140 pixels, the title and the paragraph are very close touching, maybe you could improve the spacing on these elements.
Otherwise, well done! Good luck for your upcoming projects and challenges.
Marked as helpful2@o0oDanielHaroo0oPosted over 3 years ago@mathcrln Hi Mathieu, thank you!
I see what you're saying about the padding, I'll try that. I'll fix the spacing as well. Thanks for the feedback!
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