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

NcInputField should be submitted when using the Enter key #5824

Open
GVodyanov opened this issue Jul 18, 2024 · 1 comment
Open

NcInputField should be submitted when using the Enter key #5824

GVodyanov opened this issue Jul 18, 2024 · 1 comment
Labels
enhancement New feature or request feature: input-field Covering the InputField, TextField, ...

Comments

@GVodyanov
Copy link
Contributor

Due to using the @trailing-button-click to submit the input, hitting the Enter key doesn't do anything, as it should in my opinion.

Also, due to the custom event, using min-length doesn't work, and you have to check whether there are any characters inputted in the triggered method.

Somewhat relevant to nextcloud/contacts#4039

@GVodyanov GVodyanov added enhancement New feature or request feature: input-field Covering the InputField, TextField, ... labels Jul 18, 2024
@ShGKme
Copy link
Contributor

ShGKme commented Jul 18, 2024

submit is an event of a form element, not an input element. So using Enter for submitting is a form's feature.

trailing-button-click doesn't always mean "submit". It might be "clear", for example.

If we want to add an ability to submit by trailing button, we may add a way to provide nativeType to NcButton of the trailing button. Then we can use submit natively.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature: input-field Covering the InputField, TextField, ...
Projects
None yet
Development

No branches or pull requests

2 participants