Clear search input on submit

[Ishaan-11]

SnapShot/src/App.js

Line 13 in 406d695

e.currentTarget.reset();

If this line is used for clearing the search input then it is not being cleared.
I suggest adding setSearchEntry(""); after handleSubmit call

SnapShot/src/components/Form.js

Line 12 in 406d695

onSubmit={e => handleSubmit(e, history, searchEntry)}

[kushalShukla-web]

ishaan can you pls tell me how this handlechange is working i dont
understand it.

[kushalShukla-web]

sorry ishaan but i can't see my committed code in ur main code. im new in opensource.

[Ishaan-11]

Hi @kushalShukla-web , this repo belongs to @Yog9 , so for your changes to reflect you need create Pull Request on the repo of @Yog9 .
I would recommend this video for open source guidance.
I hope this helps you.

[SOMKUVAR]

I want to work on this issue.
Please assign me.

[Dhruva21]

Hi @Yog9 ,
I have a question regarding the handle submit function which is written in App.js. We initialized the searchEntry state in Form.js and just sending the updated searchEntry to handle submit function. But we cannot update the state and clear the search bar. But instead we can control handle submit function in Form.js and update the searchEntry state.
My fix: I removed the handleSubmit prop in

and removed the handle submit function from App.js and handle that in Form.js. Is this the correct fix? Or anything else is required, please let me know.
Code change in App.js:

**Code change in Form.js:** const handleSubmit = (e, history, searchEntry) => { e.preventDefault(); e.currentTarget.reset(); let url = `/search/${searchEntry}`; history.push(url); **setSearchEntry('')** };