FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: add number & phone inputType for prompt dialog by shyamseshadri · Pull Request #6365 · NativeScript/NativeScript · GitHub

feat: add number & phone inputType for prompt dialog - #6365

Merged
manoldonev merged 10 commits into
NativeScript:masterfrom
shyamseshadri:issue-6363-dialog-prompt-number
Nov 7, 2018
Merged

manoldonev merged 10 commits into
NativeScript:masterfrom
shyamseshadri:issue-6363-dialog-prompt-number

Conversation

shyamseshadri commented Oct 5, 2018
edited by vakrilov
Loading

Copy link
Copy Markdown
Contributor

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 added the ♥ community PR label Oct 5, 2018
ns-bot added the cla: yes label Oct 5, 2018
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 assigned vakrilov Oct 5, 2018
ghost added in progress and removed ♥ community PR labels Oct 5, 2018

vakrilov commented Oct 5, 2018

Copy link
Copy Markdown
Contributor

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 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 force-pushed the issue-6363-dialog-prompt-number branch from 56ba7ea to f164f1b Compare October 6, 2018 04:20

Copy link
Copy Markdown
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)

Copy link
Copy Markdown
Contributor Author

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

vakrilov commented Oct 8, 2018

Copy link
Copy Markdown
Contributor

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

vakrilov commented Oct 8, 2018

Copy link
Copy Markdown
Contributor

test

MartoYankov added the docs needed Additional documentation on this issue/PR is needed label Oct 8, 2018

Copy link
Copy Markdown
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 assigned manoldonev Nov 6, 2018

Copy link
Copy Markdown
Contributor

test

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

Copy link
Copy Markdown
Contributor

test

Copy link
Copy Markdown
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 merged commit 7e7c050 into NativeScript:master Nov 7, 2018
ghost removed the in progress label Nov 7, 2018

radeva commented Nov 19, 2018

Copy link
Copy Markdown

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 Bot commented Jan 11, 2020

Copy link
Copy Markdown

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 Bot locked and limited conversation to collaborators Jan 11, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL