Merge pull request Expensify#26584 from allroundexperts/fix-26579

fix: do not reset the state if same tab is clicked
allroundexperts · Sep 3, 2023 · d0b2772 · d0b2772
2 parents e088539 + 3e441a5
commit d0b2772
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/components/TabSelector/TabSelector.js b/src/components/TabSelector/TabSelector.js
@@ -98,13 +98,17 @@ function TabSelector({state, navigation, onTabPress, position}) {
                 const isFocused = index === state.index;
 
                 const onPress = () => {
+                    if (isFocused) {
+                        return;
+                    }
+
                     const event = navigation.emit({
                         type: 'tabPress',
                         target: route.key,
                         canPreventDefault: true,
                     });
 
-                    if (!isFocused && !event.defaultPrevented) {
+                    if (!event.defaultPrevented) {
                         // The `merge: true` option makes sure that the params inside the tab screen are preserved
                         navigation.navigate({name: route.name, merge: true});
                     }