-
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
[Touchable* + Text] "Touchable cannot transition" error about 20% of the time #1693
Comments
+1 |
This happens for me with regularity but only when running on the device and debugging in Chrome. If I run on the device without debugging this error does not occur. Anyone seeing the same behavior? |
@spicyj - assigned you because it seems related to responders 😄 |
I don't actually know anything about this stuff… maybe @vjeux does. |
I've been getting the same error with a TouchableHighlight wrapping an Image, so it doesn't seem related to Text. I've only seen this while debugging in Chrome thus far but will update if that changes. |
It is same when i'm wrapping the |
@brentvatne @vjeux @spicyj The same error comes up when we try to use inspect element mode. It seems the elements there are also |
?? |
I had the same issue with Image surrounded by TouchableHighlight. Also just coming up on a real device while debugging in Chrome. Thanks uoroaix. Surrounding the Image with View solved the problem. |
Do we have an official fix or explanation ? This error is so annoying ! |
Any update on this, @vjeux? The error message is cryptic and the stack trace is useless to us. |
👍 |
No update sorry, if anyone of you can investigate what is happening and figure out if you can make a fix that would be awesome! |
For reference, I was seeing a heap of these warnings, and it turned out I had removed the LaunchScreen from the project (also fixed #2018 for me). No idea why the LaunchScreen would be so important, but it fixed if for me. |
Also happening on Android, real device. |
Seeing this too on Nexus 6 + Chromium debugging. App is just a very simple test of Navigator, and it works without debugging, though route transitions are sluggish. As soon as I debug it, I get a red screen of death with the above message. |
@RoryHunter Check https://facebook.github.io/react-native/docs/performance.html#slow-navigator-transitions for the transitions issue |
Encountered this same issue:
Only started happening to me this morning, but went through the following steps to resolve it:
Given that the restart and some further fiddling resolved the issue, could this be memory related? Here's further details: Stack Trace
Component source: <TouchableHighlight
style={styles.container}
underlayColor='#eee'
onPress={this.props.handler}>
<View style={styles.container}>
<View style={styles.label}>
<Text style={styles.labelText}>{this.props.label}</Text>
</View>
<View style={styles.icon}>
<Icon name={this.props.icon} size={60} color="green" />
</View>
</View>
</TouchableHighlight> Inspector screenshot:Hope any of this helps! |
+1 I have been getting this a lot lately. |
👍 same here |
👍 have been encountering repeatedly. |
+1 |
1 similar comment
+1 |
I'm not sure if this will help, but I had the same error message (red box of death) that would sometimes pop up on my emulator and always cause my physical device to show a blank page. I wasn't getting the log information needed via my emulator, so I realized I could get more from plugging in my device and logging out everything that is going on via Android Studio's "Logcat". In there I found that there was an issue with the way I was using componentDidMount (specifically "E/ReactNativeJS: TypeError: undefined is not an object (evaluating 'a.componentDidMount.bind')" ). While your error may be different, seeing a very verbose log from your device may prove to be beneficial in narrowing down the issue :) |
Hey, everyone, this looks like this a developer experience issue and is a bit of a low priority for us. We are probably not going to tackle this soon, but; the ideal solution most likely would be that this should be a warning here instead of a thrown exception. We'd encourage sending a pull request for this issue. |
I can confirm that this issue is still happening on RN 0.60.3 on Android, Samsung Galaxy S7 |
+1, xiaomi redmi 5 plus 🤷♂️ |
Hi All I had the same annoying error during debugging on a v0.57 app but I managed to hack/fix it. This error occurs in node_modules/react-native/Libraries/Components/Touchable/touchable.js For me, you get
on line 666 of all places! This only gets called by touchableHandleResponderGrant() So I just decided to remove/comment out the lines that called it:
Yes, this is a hack but I wanted to debug properly so I needed that error to go away. If your app requires long presses I assume something will break, but then I think the new pressables in RN will supercede this anyway. Hope this helps some people! o/ |
Summary: We realized lately that the bot has "too much to eat" and because by default it starts from the most recent things ([see default here](https://github.com/actions/stale#ascending)), I'm fixing the settings so that it would start from oldest. This should make it start from [very old and inactive issues](https://github.com/facebook/react-native/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-asc+) such as such as #1693 (last comment from Aug 2020) or #10027 (last activity in May 2019) and make it do significant progress on the repo clean up :) ## Changelog <!-- Help reviewers and the release process by writing your own changelog entry. Pick one each for the category and type tags: [ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message For more details, see: https://reactnative.dev/contributing/changelogs-in-pull-requests --> [INTERNAL] [CHANGED] - set the stale bot to start from oldest Pull Request resolved: #36337 Test Plan: N/A Reviewed By: cortinico Differential Revision: D43694724 Pulled By: cipolleschi fbshipit-source-id: a7cc07e4669b1d7f5e0bfc31ff313767790e1f0f
This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
Summary: We realized lately that the bot has "too much to eat" and because by default it starts from the most recent things ([see default here](https://github.com/actions/stale#ascending)), I'm fixing the settings so that it would start from oldest. This should make it start from [very old and inactive issues](https://github.com/facebook/react-native/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-asc+) such as such as facebook#1693 (last comment from Aug 2020) or facebook#10027 (last activity in May 2019) and make it do significant progress on the repo clean up :) ## Changelog <!-- Help reviewers and the release process by writing your own changelog entry. Pick one each for the category and type tags: [ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message For more details, see: https://reactnative.dev/contributing/changelogs-in-pull-requests --> [INTERNAL] [CHANGED] - set the stale bot to start from oldest Pull Request resolved: facebook#36337 Test Plan: N/A Reviewed By: cortinico Differential Revision: D43694724 Pulled By: cipolleschi fbshipit-source-id: a7cc07e4669b1d7f5e0bfc31ff313767790e1f0f
Anybody reproduces this issue on 0.73? I will implement the solution discussed #1693 (comment) Thanks |
I think this problem is obsolete and can be closed. I can confirm with 100% certainty that this problem does not occur, and I have not encountered anything like this since version 60 |
There hasn't been any recent activity on this issue for a while. Additionally, recent conversations suggest that it is no longer reproducible. Based on the findings shared above, it seems to occur mostly while debugging on Chrome. I am closing this issue for now. If anyone experiences this issue again, please feel free to reopen it or create a new issue with a valid reproducer using this template and steps to reproduce the problem. |
Hi I am not really sure if this is an issue or not, but I thought i posted here so if anyone have the same problem, they know what was going on.
Apparently if you render some touchablebutton to be like this
you will run into this error very frequently
To solve the problem just wrap the text inside the
touchableopacity/touchablehighlight with a child view like this
and seems to not run into that error anymore.
The text was updated successfully, but these errors were encountered: