Skip to content

[TextInput] Add onLayout to TextInput, forwarding it to the native views #1296

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

Closed
wants to merge 1 commit into from

Conversation

ide
Copy link
Contributor

@ide ide commented May 15, 2015

TextInput does not automatically forward all props using the spread operator so we need to explicitly forward the onLayout prop.

Test Plan: Mount a TextInput component with an onLayout prop and see that the callback handler is invoked with the TextInput's frame.

…views

`TextInput` does not automatically forward all props using the spread operator so we need to explicitly forward the `onLayout` prop.

Test Plan: Mount a TextInput component with an `onLayout` prop and see that the callback handler is invoked with the TextInput's frame.
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 15, 2015
@vjeux
Copy link
Contributor

vjeux commented May 15, 2015

@facebook-github-bot import

@vjeux vjeux self-assigned this May 15, 2015
@facebook-github-bot
Copy link
Contributor

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

@ide ide closed this in d720459 May 15, 2015
@ide ide deleted the layout-text-input branch May 19, 2015 02:22
ryanlntn pushed a commit to ryanlntn/react-native that referenced this pull request Aug 9, 2022
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