Skip to content

Conversation

@shyamseshadri
Copy link
Contributor

@shyamseshadri shyamseshadri commented Oct 5, 2018

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 .

@ghost ghost added the ♥ community PR label Oct 5, 2018
@ns-bot ns-bot added the cla: yes label Oct 5, 2018
@NickIliev NickIliev added ready for test TSC needs to test this and confirm against live production apps and automated test suites and removed ready for test TSC needs to test this and confirm against live production apps and automated test suites labels Oct 5, 2018
@ghost ghost assigned vakrilov Oct 5, 2018
@ghost ghost added in progress and removed ♥ community PR labels Oct 5, 2018
@vakrilov
Copy link
Contributor

vakrilov commented Oct 5, 2018

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.

@shyamseshadri shyamseshadri changed the title feat: Add number inputType for prompt dialog feat: Add number & phone inputType for prompt dialog Oct 6, 2018
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.
@shyamseshadri shyamseshadri force-pushed the issue-6363-dialog-prompt-number branch from 56ba7ea to f164f1b Compare October 6, 2018 04:20
@shyamseshadri
Copy link
Contributor Author

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)

@shyamseshadri
Copy link
Contributor Author

hey @vakrilov , let me know if this looks good or if you need me to make any other changes.

@vakrilov
Copy link
Contributor

vakrilov commented Oct 8, 2018

Thanks!
Tests and code looks good. I will inform you if there is any action needed from your side.

@vakrilov
Copy link
Contributor

vakrilov commented Oct 8, 2018

test

@MartoYankov MartoYankov added the docs needed Additional documentation on this issue/PR is needed label Oct 8, 2018
@vakrilov
Copy link
Contributor

Hey @shyamseshadri - thanks again for the PR!
We are currently in a code-freeze phase for the NativeScript 5.0 release. This means it might take more time to merge PRs into master.

We will notify you if the is some action needed form your side.
Thanks for understanding and being awesome part of the NS community!

@ghost ghost assigned manoldonev Nov 6, 2018
@manoldonev
Copy link
Contributor

test

@manoldonev manoldonev changed the title feat: Add number & phone inputType for prompt dialog feat: add number & phone inputType for prompt dialog Nov 6, 2018
@manoldonev
Copy link
Contributor

test

@manoldonev
Copy link
Contributor

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/

@manoldonev manoldonev merged commit 7e7c050 into NativeScript:master Nov 7, 2018
@ghost ghost removed the in progress label Nov 7, 2018
@radeva
Copy link

radeva commented Nov 19, 2018

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: 
👉Your full name 
👉Your email 
👉Your country or residence 

Best regards,
The NativeScript Team

@lock
Copy link

lock bot commented Jan 11, 2020

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.

@lock lock bot locked and limited conversation to collaborators Jan 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: New Dialog Prompt Input Type for Phone / Numbers

8 participants