Skip to content

Add keyboardAppearance prop to TextInput component.#4012

Closed
christopherdro wants to merge 1 commit into
react:masterfrom
christopherdro:keyboardAppearance
Closed

Add keyboardAppearance prop to TextInput component.#4012
christopherdro wants to merge 1 commit into
react:masterfrom
christopherdro:keyboardAppearance

Conversation

@christopherdro

Copy link
Copy Markdown
Contributor

Resolves #3649.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

By analyzing the blame information on this pull request, we identified @nicklockwood, @sahrens and @brentvatne to be potential reviewers.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Nov 9, 2015
Comment thread React/Base/RCTConvert.m Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

style nit: unindent 2 spaces

@ide

ide commented Nov 9, 2015

Copy link
Copy Markdown
Contributor

Looks good and is consistent with the other APIs. If you have time, could you add an example to the UIExplorer too?

@nicklockwood

Copy link
Copy Markdown
Contributor

As @ide says, an example would be great. Other than that, looks good to me.

@christopherdro

Copy link
Copy Markdown
Contributor Author

@ide @nicklockwood Done.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

@christopherdro updated the pull request.

@ide

ide commented Nov 9, 2015

Copy link
Copy Markdown
Contributor

Thanks!

@nicklockwood

Copy link
Copy Markdown
Contributor

@facebook-github-bot import

@facebook-github-bot

Copy link
Copy Markdown
Contributor

Thanks for importing. If you are an FB employee go to https://our.intern.facebook.com/intern/opensource/github/pull_request/778005025655512/int_phab to review.

@ghost ghost closed this in f407211 Nov 11, 2015
@nicklockwood

Copy link
Copy Markdown
Contributor

Note: I removed the "alert" style, as it's deprecated.

ide pushed a commit to expo/react-native that referenced this pull request Nov 13, 2015
Summary: Resolves react#3649.
Closes react#4012

Reviewed By: javache

Differential Revision: D2636538

Pulled By: nicklockwood

fb-gh-sync-id: 022e79d8f8fa684cad43af1a51f728d60ac652a8
sunnylqm pushed a commit to sunnylqm/react-native that referenced this pull request Dec 2, 2015
Summary: Resolves react#3649.
Closes react#4012

Reviewed By: javache

Differential Revision: D2636538

Pulled By: nicklockwood

fb-gh-sync-id: 022e79d8f8fa684cad43af1a51f728d60ac652a8
Crash-- pushed a commit to Crash--/react-native that referenced this pull request Dec 24, 2015
Summary: Resolves react#3649.
Closes react#4012

Reviewed By: javache

Differential Revision: D2636538

Pulled By: nicklockwood

fb-gh-sync-id: 022e79d8f8fa684cad43af1a51f728d60ac652a8
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants