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

navigation.dispatch() works flaky on iOS (new arch) #1949

Open
tboba opened this issue Oct 27, 2023 · 0 comments
Open

navigation.dispatch() works flaky on iOS (new arch) #1949

tboba opened this issue Oct 27, 2023 · 0 comments
Labels
Platform: iOS This issue is specific to iOS Repro provided A reproduction with a snack or repo is provided

Comments

@tboba
Copy link
Member

tboba commented Oct 27, 2023

Description

When I was testing #1738, I've spotted a weird bug while using navigation.dispatch() with type RESET on iOS.
It seems that when user dispatches a reset to the navigation.dispatch, the screen is not changed - moreover, when orientation is also being changed, this view is being skewed during the rotation, resulting view to be narrowed:

Screen.Recording.2023-10-27.at.15.49.34.mov

As I have discovered, this issue should be reproducible on iOS 17, I've also tested it on iOS 16.

Steps to reproduce

  1. Launch repro from snack attached below. This repro comes originally from Incorrect layout when navigating between screens of different orientation on iOS #1738, but apparently it works also there
  2. Press the button to go to landscape screen
  3. Press the button to go to home screen (it should have portrait orientation)
  4. The view should be narrowed

Snack or a link to a repository

https://github.com/tboba/screens-repro/tree/%40tboba/rns-orientation-glitch

Screens version

3.27.0

React Native version

0.72.4

Platforms

iOS

JavaScript runtime

Hermes

Workflow

None

Architecture

Fabric (New Architecture)

Build type

None

Device

iOS simulator

Device model

iPhone 15 Pro (iOS 17.0)

Acknowledgements

Yes

@github-actions github-actions bot added Missing repro This issue need minimum repro scenario Platform: iOS This issue is specific to iOS labels Oct 27, 2023
@tboba tboba added Repro provided A reproduction with a snack or repo is provided and removed Missing repro This issue need minimum repro scenario labels Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Platform: iOS This issue is specific to iOS Repro provided A reproduction with a snack or repo is provided
Projects
None yet
Development

No branches or pull requests

1 participant