Skip to content

Commit ad29b3c

Browse files
authored
Merge pull request Expensify#63826 from Expensify/revert-63242-perf/navigation-tab-bar-dummy
[CP Staging] Revert "NavigationTabBar: Add dummy component"
2 parents 145beee + a983c73 commit ad29b3c

7 files changed

Lines changed: 18 additions & 317 deletions

File tree

src/components/Navigation/NavigationTabBar/NavigationTabBarDummy.tsx

Lines changed: 0 additions & 299 deletions
This file was deleted.

src/components/Navigation/SearchSidebar.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ import CONST from '@src/CONST';
1717
import ONYXKEYS from '@src/ONYXKEYS';
1818
import SCREENS from '@src/SCREENS';
1919
import type {SearchResults} from '@src/types/onyx';
20+
import NavigationTabBar from './NavigationTabBar';
2021
import NAVIGATION_TABS from './NavigationTabBar/NAVIGATION_TABS';
21-
import NavigationTabBarDummy from './NavigationTabBar/NavigationTabBarDummy';
2222
import TopBar from './TopBar';
2323

2424
type SearchSidebarProps = {
@@ -76,7 +76,7 @@ function SearchSidebar({state}: SearchSidebarProps) {
7676
/>
7777
<SearchTypeMenu queryJSON={queryJSON} />
7878
</View>
79-
<NavigationTabBarDummy selectedTab={NAVIGATION_TABS.SEARCH} />
79+
<NavigationTabBar selectedTab={NAVIGATION_TABS.SEARCH} />
8080
</View>
8181
);
8282
}

src/pages/Search/SearchPageNarrow.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@ import Animated, {clamp, runOnJS, useAnimatedScrollHandler, useAnimatedStyle, us
55
import FullPageNotFoundView from '@components/BlockingViews/FullPageNotFoundView';
66
import type {DropdownOption} from '@components/ButtonWithDropdownMenu/types';
77
import HeaderWithBackButton from '@components/HeaderWithBackButton';
8+
import NavigationTabBar from '@components/Navigation/NavigationTabBar';
89
import NAVIGATION_TABS from '@components/Navigation/NavigationTabBar/NAVIGATION_TABS';
9-
import NavigationTabBarDummy from '@components/Navigation/NavigationTabBar/NavigationTabBarDummy';
1010
import TopBar from '@components/Navigation/TopBar';
1111
import ScreenWrapper from '@components/ScreenWrapper';
1212
import Search from '@components/Search';
@@ -141,7 +141,7 @@ function SearchPageNarrow({queryJSON, headerButtonsOptions, currentSearchResults
141141
testID={SearchPageNarrow.displayName}
142142
shouldEnableMaxHeight
143143
offlineIndicatorStyle={styles.mtAuto}
144-
bottomContent={<NavigationTabBarDummy selectedTab={NAVIGATION_TABS.SEARCH} />}
144+
bottomContent={<NavigationTabBar selectedTab={NAVIGATION_TABS.SEARCH} />}
145145
headerGapStyles={styles.searchHeaderGap}
146146
shouldShowOfflineIndicator={!!searchResults}
147147
>

src/pages/home/sidebar/BaseSidebarScreen.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import React, {useEffect} from 'react';
22
import {View} from 'react-native';
3+
import NavigationTabBar from '@components/Navigation/NavigationTabBar';
34
import NAVIGATION_TABS from '@components/Navigation/NavigationTabBar/NAVIGATION_TABS';
4-
import NavigationTabBarDummy from '@components/Navigation/NavigationTabBar/NavigationTabBarDummy';
55
import TopBar from '@components/Navigation/TopBar';
66
import ScreenWrapper from '@components/ScreenWrapper';
77
import useLocalize from '@hooks/useLocalize';
@@ -27,7 +27,7 @@ function BaseSidebarScreen() {
2727
shouldEnableKeyboardAvoidingView={false}
2828
style={[styles.sidebar, isMobile() ? styles.userSelectNone : {}]}
2929
testID={BaseSidebarScreen.displayName}
30-
bottomContent={!shouldDisplayLHB && <NavigationTabBarDummy selectedTab={NAVIGATION_TABS.HOME} />}
30+
bottomContent={!shouldDisplayLHB && <NavigationTabBar selectedTab={NAVIGATION_TABS.HOME} />}
3131
>
3232
{({insets}) => (
3333
<>
@@ -39,7 +39,7 @@ function BaseSidebarScreen() {
3939
<View style={[styles.flex1]}>
4040
<SidebarLinksData insets={insets} />
4141
</View>
42-
{shouldDisplayLHB && <NavigationTabBarDummy selectedTab={NAVIGATION_TABS.HOME} />}
42+
{shouldDisplayLHB && <NavigationTabBar selectedTab={NAVIGATION_TABS.HOME} />}
4343
</>
4444
)}
4545
</ScreenWrapper>

src/pages/settings/InitialSettingsPage.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ import CustomStatusBarAndBackgroundContext from '@components/CustomStatusBarAndB
1313
import Icon from '@components/Icon';
1414
import * as Expensicons from '@components/Icon/Expensicons';
1515
import MenuItem from '@components/MenuItem';
16+
import NavigationTabBar from '@components/Navigation/NavigationTabBar';
1617
import NAVIGATION_TABS from '@components/Navigation/NavigationTabBar/NAVIGATION_TABS';
17-
import NavigationTabBarDummy from '@components/Navigation/NavigationTabBar/NavigationTabBarDummy';
1818
import {PressableWithFeedback} from '@components/Pressable';
1919
import ScreenWrapper from '@components/ScreenWrapper';
2020
import {ScrollOffsetContext} from '@components/ScrollOffsetContextProvider';
@@ -422,7 +422,7 @@ function InitialSettingsPage({currentUserPersonalDetails}: InitialSettingsPagePr
422422
<ScreenWrapper
423423
includeSafeAreaPaddingBottom
424424
testID={InitialSettingsPage.displayName}
425-
bottomContent={!shouldDisplayLHB && <NavigationTabBarDummy selectedTab={NAVIGATION_TABS.SETTINGS} />}
425+
bottomContent={!shouldDisplayLHB && <NavigationTabBar selectedTab={NAVIGATION_TABS.SETTINGS} />}
426426
shouldEnableKeyboardAvoidingView={false}
427427
>
428428
{headerContent}
@@ -455,7 +455,7 @@ function InitialSettingsPage({currentUserPersonalDetails}: InitialSettingsPagePr
455455
}}
456456
/>
457457
</ScrollView>
458-
{shouldDisplayLHB && <NavigationTabBarDummy selectedTab={NAVIGATION_TABS.SETTINGS} />}
458+
{shouldDisplayLHB && <NavigationTabBar selectedTab={NAVIGATION_TABS.SETTINGS} />}
459459
</ScreenWrapper>
460460
);
461461
}

0 commit comments

Comments
 (0)