Vanilla JS/CSS

Community feedback

• 

  Mohammed Fakih•1,590

  @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 helpful

  0

  ---

  

  Rob Simpson•490

  @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

  

  Rob Simpson•490

  @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

  

  Mohammed Fakih•1,590

  @javascriptor1Posted over 1 year ago

  Hi Rob @pertrai1 ,

  Great work. It's fixed.

  Keep Coding 🚀

  Marked as helpful

  0