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

Correct dialog form submission with enter key #1378

Merged

Conversation

andreassteinmann
Copy link
Collaborator

@andreassteinmann andreassteinmann commented Feb 24, 2023

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

Forms in dialogs (modals) cannot be submitted using the "Enter" key if the form includes more than one form field.
Reason: The submit button is positioned outside the form tag.

What Is the New Behavior?

Forms in dialogs (modals) can now be submitted using the "Enter" key.

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

AB#84027

…dd correct submit button type, remove click event from submit button
@andreassteinmann andreassteinmann self-assigned this Feb 24, 2023
@andreassteinmann andreassteinmann marked this pull request as draft February 27, 2023 07:10
@SGrueber SGrueber removed the request for review from shauke February 27, 2023 10:45
@SGrueber SGrueber marked this pull request as ready for review February 27, 2023 10:55
@SGrueber SGrueber added this to the 4.0 milestone Feb 27, 2023
@andreassteinmann andreassteinmann merged commit b60e0a1 into develop Feb 27, 2023
@andreassteinmann andreassteinmann deleted the fix/correct-dialog-form-submission-with-enter-key branch February 27, 2023 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants