From 0a8681176b0f2dd0fe7dbab37dde41a77eccc3fa Mon Sep 17 00:00:00 2001 From: Brent Vatne Date: Tue, 30 Oct 2018 18:03:32 -0700 Subject: [PATCH] Merge pull request #16 from seanmorton/@seanmorton/no_more_stack_actions_reset Don't use a reset sub-action for drawer items --- packages/drawer/src/views/DrawerSidebar.js | 18 +----------------- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/packages/drawer/src/views/DrawerSidebar.js b/packages/drawer/src/views/DrawerSidebar.js index 04c03309..bbd2e3e5 100644 --- a/packages/drawer/src/views/DrawerSidebar.js +++ b/packages/drawer/src/views/DrawerSidebar.js @@ -45,24 +45,8 @@ class DrawerSidebar extends React.PureComponent { _onItemPress = ({ route, focused }) => { if (!focused) { - let subAction; - // TODO (v3): Revisit and repeal this behavior: - // if the child screen is a StackRouter then always navigate to its first screen (see #1914) - if (route.index != null && route.index !== 0) { - subAction = StackActions.reset({ - index: 0, - actions: [ - NavigationActions.navigate({ - routeName: route.routes[0].routeName, - }), - ], - }); - } this.props.navigation.dispatch( - NavigationActions.navigate({ - routeName: route.routeName, - action: subAction, - }) + NavigationActions.navigate({ routeName: route.routeName }) ); } };