-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat: add number & phone inputType for prompt dialog #6365
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
feat: add number & phone inputType for prompt dialog #6365
Conversation
|
Hey @shyamseshadri, Thanks for the PR! The code looks good. Would you mind adding a showcase of the new option along with the other prompt types in this test page. |
Add an inputType number to the PromptOptions for dialogs Closes NativeScript#6363
Add support for phone inputType to prompt dialog, and add both prompt options to the dialogs test page.
56ba7ea to
f164f1b
Compare
|
Added the tests, and also added inputType phone in additio to number. There was one unit test failing, but I'm pretty sure it was unrelated to my code Test: LISTVIEW.test_scrollToIndex_should_coerce_larger_index_to_last_item_index FAILED: last element is visible Actual: (boolean). Expected: (boolean) |
|
hey @vakrilov , let me know if this looks good or if you need me to make any other changes. |
|
Thanks! |
|
test |
|
Hey @shyamseshadri - thanks again for the PR! We will notify you if the is some action needed form your side. |
|
test |
|
test |
|
merging this -- agreed with @SvetoslavTsenov that he will update the images for the failing ui dialog tests: http://nsbuild01.telerik.com:8080/build/job/pr-composite-ns-core-modules/733/ |
|
Hi @shyamseshadri , Congratulations on being one of the winners in the {N} First-time contributors contest! You can claim your prize by contacting us at nativescriptwinners[at]progress.com not later than Nov 30th 2018 . Make sure you send us the following info: Best regards, |
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Add an inputType number to the PromptOptions for dialogs
Closes #6363
PR Checklist
What is the current behavior?
This is to support #6363
Currently, NativeScript only allows accepting inputType email, phone and password to prompt dialog.
What is the new behavior?
This change adds a inputType number which shows the numeric pad in android and iOS for dialog prompts
Fixes/Implements/Closes #6363 .