-
Notifications
You must be signed in to change notification settings - Fork 106
add defaultdata to submitDialogForm #57
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
Conversation
Adding content of prompt even for regular form submit outside clicks e.g. by clicking enter
I also did not build and commit (only built locally to check if it works). I seriously have no idea about the test and deploy process here 😆 |
Thanks @fr-ser 🙌 |
Also... Tests were broken for long, they should run now. |
It looks like the fix never made it into the dist folder in version 1.4.1.
but
|
Thanks for @jochemmeyers |
This doesn't work even now in v1.4.1. How can we fix this issue? I mean if I don't enter any string in input and hit enter then it returns an empty string, that's fine but as soon as I enter some text and hit enter then it returns an object with data property as null. |
Looking into this. |
constantly looking when it will get updated on npm bcz current project needs submitDialogForm. |
@and-ocean2017 package has been updated on npm. Please let me know if new version resolves the issue so i can close #52 |
yeah @Godofbrowser it resolves the null issue 👍, but I think it would be more convenient to show some kind of error if data property is an empty string (i.e if the user hit enter as soon as prompt shows without inserting data). There should be a kind of option maybe showEmptyError just to check if the user inserted data and hit enter then we are good to go else if data is an empty string then show error without closing the dialog box. So that user can inset data again.
|
Adding the content of prompt even for regular form submit outside clicks
e.g. by clicking enter
The one-line change seems pretty obvious. I couldn't get the tests running on my machine otherwise I would have added a test.
Solves this bug: #52