Skip to content

Commit

Permalink
avoid unnecessary navigation completion dispatches
Browse files Browse the repository at this point in the history
  • Loading branch information
ericvicenti committed Apr 4, 2018
1 parent d57d118 commit d2fe180
Showing 1 changed file with 12 additions and 7 deletions.
19 changes: 12 additions & 7 deletions src/views/StackView/StackView.js
Expand Up @@ -25,14 +25,19 @@ class StackView extends React.Component {
navigation={this.props.navigation}
descriptors={this.props.descriptors}
onTransitionStart={this.props.onTransitionStart}
onTransitionEnd={(lastTransition, transition) => {
onTransitionEnd={(transition, lastTransition) => {
const { onTransitionEnd, navigation } = this.props;
navigation.dispatch(
StackActions.completeTransition({
key: navigation.state.key,
})
);
onTransitionEnd && onTransitionEnd(lastTransition, transition);
if (
transition.navigation.state.isTransitioning &&
!lastTransition.navigation.state.isTransitioning
) {
navigation.dispatch(
StackActions.completeTransition({
key: navigation.state.key,
})
);
}
onTransitionEnd && onTransitionEnd(transition, lastTransition);
}}
/>
);
Expand Down

0 comments on commit d2fe180

Please sign in to comment.