Skip to content
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

add all keyboard types for keyboardType enum prop #474

Closed
wants to merge 1 commit into from
Closed

add all keyboard types for keyboardType enum prop #474

wants to merge 1 commit into from

Conversation

shrugs
Copy link

@shrugs shrugs commented Mar 30, 2015

This adds the entire suite of keyboardTypes to the enum.

I matched the enums with the constants (UIKeyboardTypeNumbersAndPunctuation becomes numbersAndPunctuation).

I also added shortcut enums for the popular and verbose names. (ascii, number, phone, etc).

@shrugs
Copy link
Author

shrugs commented Mar 30, 2015

Just realized that this might not be ideal because some of these keyboard types are iOS-specific. What's the correct way to implement these while separating platforms? Is it to subclass TextInput and make TextInputIOS that supports these different styles, along with the returnKeyType property?

@a2
Copy link
Contributor

a2 commented Apr 1, 2015

This was added in #265.

@a2 a2 closed this Apr 1, 2015
ayushjainrksh referenced this pull request in MLH-Fellowship/react-native Jul 2, 2020
* Added missing code

The example was not working at all. There was a lot of code missing in order to make it work. I added the missing code.

* Added one more line

Forgot `new MainReactPackage()` in `MainApplication.java`

* Changed singlequotes to double quotes in java
ayushjainrksh referenced this pull request in MLH-Fellowship/react-native Jul 2, 2020
tido64 pushed a commit to tido64/react-native that referenced this pull request Jul 8, 2020
* Fixing OS specific failure in publish task

* Adding publish verfification into the PR job

* Adding some comments.

* Removing the diagnostic logs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants