Design comparison
SolutionDesign
Solution retrospective
What are you most proud of, and what would you do differently next time?
- I'm proud that I did email validation in another project so I copied the logic in this project
- I also aligned the elements using CSS Grid, instead of Flexbox which I'm used to
- I used two components for the default state and success state
- Passing data from the parent component (App.jsx) to the child component (Success.jsx) was tricky but with research, it became straightforward
Community feedback
- @yannanclsPosted about 2 months ago
You should probably improve your code's responsiveness; as it stands, when the screen width reduces, the photo on the left overlaps the texts on the right. Furthermore, when the maximum width is around 1000px, the screen design appears disordered. But overall, a job well done!
Marked as helpful0@duncan-chegePosted about 2 months ago@yannancls Thank you! I just checked out that the tablet view needs more work. Let me fix it
0@duncan-chegePosted 26 days ago@yannancls I've fixed the responsiveness on smaller screens. Please check it out and let me know
0@yannanclsPosted 26 days ago@duncan-chege I looked into it and it is good. The feedbacks I listed are now fixed. Thank you!
Marked as helpful0
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