feat: add number & phone inputType for prompt dialog#6365
feat: add number & phone inputType for prompt dialog#6365manoldonev merged 10 commits intoNativeScript:masterfrom
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 .