-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
TouchableOpacity only supports a single child #1354
Comments
this may just be documentation error, so if this is expected I can make a PR for the doc update. |
Probably is a docs error. In the meantime you can wrap the inner elements in a view. I believe that the opacity is applied to that single child rather than cycling though each sub child element. |
@pwmckenna - is there somewhere in the documentation where it's mentioned that you can use multiple children? If so, we should definitely fix that 😄 |
@brentvatne it doesn't say either way, but its pretty adamant about only using one in practice. I'm not sure how the styling is applied if you wrap everything in a |
+1 to updating the docs |
@pwmckenna - haha good point, do you have time to put together a PR that updates the docs to indicate this? |
sure, i'll give it a go. I've been meaning to take a peak at that other doc issue as well. I'll see if I can't do both while I'm in there. |
the following code triggers fails an invariant check:
The text was updated successfully, but these errors were encountered: