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

fix: pager view recycling crash #970

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JoniVR
Copy link

@JoniVR JoniVR commented Feb 13, 2025

Summary

This pull request resolves #946.
It's also related to #859, since the stacktrace is the same.

After 6.6.1 release, I was still facing this issue and getting regular errors in crashlytics from our users.

After some investigation, it turns out that we can reproduce this when nesting a react-navigation material top tabs navigator inside a native stack navigator (react-native-screens).

After deploying this as a patch via patch-package, the exception (which happened on a lot) went away completely with no visible side-effects. We've been monitoring the fixed version for a week now.

Test Plan

What's required for testing (prerequisites)?

react-native-pager-view nested in a stack.

What are the steps to reproduce (after prerequisites)?

Unfortunately, I can't reproduce consistently yet outside my project, and I can't share the code of my original project where this is happening.

Compatibility

OS Implemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)

@JoniVR
Copy link
Author

JoniVR commented Feb 13, 2025

@MrRefactor Do you have a way to consistently reproduce this exception?

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.

[Android] App crashes on navigating out of the screen with Pager
1 participant