Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Form onSubmit should accept HTMLFormElement event #2500

Closed
corydeppen opened this issue Feb 6, 2018 · 2 comments
Closed

Form onSubmit should accept HTMLFormElement event #2500

corydeppen opened this issue Feb 6, 2018 · 2 comments

Comments

@corydeppen
Copy link
Contributor

It's not a bug, but rather a request to change the TS definition for the event parameter for onSubmit from React.FormEvent<HTMLElement> to React.FormEvent<HTMLFormElement> in the Form component. This seems more appropriate and will allow it to handle events from other libraries like Formik without a compilation error. I don't mind submitting the PR to make this quick change if there is agreement.

@welcome
Copy link

welcome bot commented Feb 6, 2018

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you've completed all the fields in the issue template so we can best help.

We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

@levithomason
Copy link
Member

Agreed, thanks!

corydeppen added a commit to corydeppen/Semantic-UI-React that referenced this issue Feb 6, 2018
levithomason pushed a commit that referenced this issue Feb 6, 2018
Brantron pushed a commit to Brantron/Semantic-UI-React that referenced this issue Mar 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants