Skip to content

feat(text-field): add closeOnReturn property to avoid auto dismissing…#8347

Merged
vmutafov merged 14 commits into
NativeScript:masterfrom
MCurran16:dismiss-input-8287
Mar 20, 2020
Merged

feat(text-field): add closeOnReturn property to avoid auto dismissing…#8347
vmutafov merged 14 commits into
NativeScript:masterfrom
MCurran16:dismiss-input-8287

Conversation

@MCurran16

Copy link
Copy Markdown
Contributor

… input on return press

PR Checklist

What is the current behavior?

The text field is automatically calling dismissSoftInput() on return pressed.

What is the new behavior?

Based on a flag called closeOnReturn, the text field will either call dismissSoftInput() or not. The closeOnReturn property is true by default so the current behavior for text fields does not change.

Fixes/Implements/Closes #8287 .

@cla-bot cla-bot Bot added the cla: yes label Feb 14, 2020
@vmutafov

Copy link
Copy Markdown
Contributor

Hey @MCurran16 ! Thank you for the PR! Could you please update the NativeScript.api.md as you have modified the public API? It is done by copying ./nativescript-core/temp/NativeScript.api.md into ./api-reports/NativeScript.api.md
There is additional information in the CONTRIBUTING.md. Thank you!

@MCurran16 MCurran16 requested a review from vmutafov March 12, 2020 15:29
@NativeScript NativeScript deleted a comment from dtopuzov Mar 20, 2020
@vmutafov vmutafov merged commit 92b5b02 into NativeScript:master Mar 20, 2020
NathanWalker pushed a commit that referenced this pull request Aug 7, 2020
#8347)

* feat(text-field): add closeOnReturn property to avoid auto dismissing input on return press

* tslint

* Update NS public api

Co-authored-by: Vasil Trifonov <v.trifonov@gmail.com>
Co-authored-by: Dimitar Topuzov <dtopuzov@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't automatically dismissSoftInput() on return pressed

4 participants