Design comparison
Community feedback
- @javascriptor1Posted over 1 year ago
Hello Rob ,
Nice work. Here is my feedback for your solution: ā¢ You have applied the mobile background image for large screens. Needs a fix. ā¢ Javascript logic is broken. Try to fill 3 input fields only then do submit. It will show an error. Then fill 4th input field which you did not fill in first try so now all fields are filled correctly then try to submit. It won't submit.
I hope this will help.
Keep coding š
Marked as helpful0@pertrai1Posted over 1 year ago@javascriptor1 thank you so much for looking at the code. I am going to make the changes for the mistakes I have made.
0@pertrai1Posted over 1 year ago@javascriptor1 - I made a change based upon your feedback. Here is the commit if you have time to look at it - https://github.com/pertrai1/frontend-challenges/commit/d901c8cf66777323deb1b36757244fef1a945256. Thanks again.
0@javascriptor1Posted over 1 year agoHi Rob @pertrai1 ,
Great work. It's fixed.
Keep Coding š
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