Design comparison
Solution retrospective
Greetings All 👋🏼,
Any feedback on anything will appreciated.
I used ::before and ::after for curves for mobile layout but failed to the same on desktop layout. I ended up using top and bottom of sections as background curves.
Regards, KT
Community feedback
- @ApplePieGiraffePosted over 3 years ago
Hi, KT! 👋
Well done on this challenge! 👍 Your solution looks nice and is responsive! 👏
I might personally actually add the curvy sections in this challenge as background images in the individual sections of the page for both the desktop and mobile layouts (since that'll keep the images in one place and make it a little easier to manage those background shapes, then). But what you have here seems to look pretty good! 🙂
One suggestion I have is to make sure that users cannot submit an empty email in the email input in the footer of the page. 😉
Keep coding (and happy coding, too)! 😁
3@KtGitItPosted over 3 years ago@ApplePieGiraffe Thank you for your feedback! I added "required" in html for email input.
1
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