Skip to content

[HOLD] TransitionEnd of react-navigation does not work properly on IOS #3283

Closed

Description

@parasharrajat

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


@marcaaron

This is a tracking issue and it is created so that we can revert to original transitionEnd event on IOS as well when this is fixed at Library level react-navigation.

Related PR: #2956

Expected Result:

transitionEnd should fire consistently for screens on IOS as well.

Actual Result:

For stack router, transitionEnd does not fire regularly on navigation screens on IOS devices. This seems to be an issue with underlying react-native-screens. This event is necessary to delay the main logic until the animation is finished to run the animation at > 50FPS.

Action Performed:

  1. Subscribe to transitionEnd event on the search page as follows:
navigation.addListener('transitionEnd', callback);
  1. Every time, when we go to searchPage, this should fire.

Workaround:

Yes, Due to the reason that react-navigation register Interactions, we can utilize InteractionManager and it is fully supported on Native devices.

Platform:

Where is this issue occurring?

Web
iOS ✅
Android
Desktop App
Mobile Web

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:

View all open jobs on Upwork

Activity

  1. MelvinBot commented on Jun 2, 2021

    @MelvinBot
    Contributor

    Triggered auto assignment to @MitchExpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

  2. removed
    AutoAssignerTriageAuto assign issues for triage to an available triage team member
    on Jun 2, 2021
  3. changed the title [-][Hold][Tracking issue] TransitionEnd of react-navigation does not work properly on IOS[/-] [+][HOLD] TransitionEnd of react-navigation does not work properly on IOS[/+] on Jun 7, 2021
  4. marcaaron commented on Jun 7, 2021

    @marcaaron
    Contributor

    This is not really actionable yet so I'm un-assigning @MitchExpensify and we can wait until there is movement at the library level...

    This is a tracking issue and it is created so that we can revert to original transitionEnd event on IOS as well when this is fixed at Library level.

    @parasharrajat can you please include details about what library we are expecting will be updated?

  5. MelvinBot commented on Nov 29, 2021

    @MelvinBot
    Contributor

    @parasharrajat, this Monthly task hasn't been acted upon in 6 weeks; closing.

    If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions