Design comparison
Solution retrospective
It was a really fun project. If improvements can be made do suggest. Thank you.
Community feedback
- @FluffyKasPosted about 3 years ago
Hey, well done on completing the challenge! it looks fine but there's a few improvements you could make!
-
Your email validation doesn't work properly, it lets my submit invalid email addresses. You should use a regex to check the input to cover most use cases ^^
-
Terms&services should be a clickable link.
-
The "pricing" and "form-container" are different widths, it would look nicer if they were the same.
-
Your button is missing a
cursor: pointer
. -
You could set a max-width on your elements instead of width. Instead of % for widths rem would be a more reliable choice.
-
You could center your container.
Overall, good job, there are just some fixes needed :)
Marked as helpful1@LahsivK4070Posted about 3 years ago@FluffyKas thanks for suggestions. Will improve the solution. Thanks
0 -
- @isatveersinghPosted about 3 years ago
you should use functions to avoid repetative code. also you can use regular expressions for email validation.
0@LahsivK4070Posted about 3 years ago@isatveersingh surely will keep that in mind. Thanks
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