Design comparison
Solution retrospective
Hey there!
I've achieved another challenge. Feel free to give me tips and tricks about JavaScript code, or on HTML and CSS if you notice anything that can be improved.
Community feedback
- @OneManBannedPosted about 1 year ago
Wisslor, well done on a really nice looking solution.
I have some suggestions for you -
- You need to add labels to your form elements. This mdn page explains how to implement them and also the additional benefits of using them.
- You need to add some space at the top and bottom of the page for mobile layouts.
- I would suggest using relative units instead of pixels this page converts them for you.
- Have a look at the validityState api. It will help shorten the code you need to write for checking form inputs.
A really nice looking solution though. Brendan.
Marked as helpful0@KevallionPosted about 1 year ago@OneManBanned
Hi Brendan !
Thank you for taking the time to write me a review. This helps me a lot, and I've learned some new things. I'm going to try to implement your suggestions in my code. Thanks also for sharing some resources with me. The pixel converters look very useful. The validityState API seems very interesting, although I don't know right now how to use it to shorten my code. Maybe in the next project, I will try to use it as much as I can.
Happy coding ! Wisllor.
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