diff --git a/native/chat/chat-thread-list.react.js b/native/chat/chat-thread-list.react.js --- a/native/chat/chat-thread-list.react.js +++ b/native/chat/chat-thread-list.react.js @@ -28,11 +28,7 @@ import type { SetState } from 'lib/types/hook-types.js'; import { threadTypes } from 'lib/types/thread-types-enum.js'; import type { ThreadInfo } from 'lib/types/thread-types.js'; -import type { - GlobalAccountUserInfo, - UserInfo, - LoggedInUserInfo, -} from 'lib/types/user-types.js'; +import type { GlobalAccountUserInfo, UserInfo } from 'lib/types/user-types.js'; import { useServerCall } from 'lib/utils/action-utils.js'; import { ChatThreadListItem } from './chat-thread-list-item.react.js'; @@ -52,11 +48,7 @@ } from '../navigation/route-names.js'; import type { TabNavigationProp } from '../navigation/tab-navigator.react.js'; import { useSelector } from '../redux/redux-utils.js'; -import { - type IndicatorStyle, - indicatorStyleSelector, - useStyles, -} from '../themes/colors.js'; +import { indicatorStyleSelector, useStyles } from '../themes/colors.js'; import type { ScrollEvent } from '../types/react-native.js'; import { AnimatedView, type AnimatedStyleObj } from '../types/styles.js'; import { animateTowards } from '../utils/animation-utils.js'; @@ -93,24 +85,13 @@ type Props = { ...BaseProps, // Redux state - +loggedInUserInfo: ?LoggedInUserInfo, - +styles: typeof unboundStyles, - +indicatorStyle: IndicatorStyle, +searchText: string, +searchStatus: SearchStatus, - +openedSwipeableID: string, +setNumItemsToDisplay: SetState, +searchCancelButtonOpen: Value, +scrollPos: { current: number }, - +onScroll: (event: ScrollEvent) => void, - +renderSearch: ( - additionalProps?: $Shape>, - ) => React.Node, +hardwareBack: () => boolean, - +renderItem: (row: { item: Item, ... }) => React.Node, - +partialListData: Item[], - +onEndReached: () => void, - +floatingAction: React.Node, + +chatThreadList: React.Node, }; class ChatThreadList extends React.PureComponent { @@ -198,39 +179,7 @@ }; render() { - let fixedSearch; - const { searchStatus } = this.props; - if (searchStatus === 'active') { - fixedSearch = this.props.renderSearch({ autoFocus: true }); - } - const scrollEnabled = - searchStatus === 'inactive' || searchStatus === 'active'; - // viewerID is in extraData since it's used by MessagePreview - // within ChatThreadListItem - const viewerID = this.props.loggedInUserInfo?.id; - const extraData = `${viewerID || ''} ${this.props.openedSwipeableID}`; - return ( - - {fixedSearch} - - {this.props.floatingAction} - - ); + return this.props.chatThreadList; } flatListRef = (flatList: ?FlatList) => { @@ -561,28 +510,71 @@ return node; }, [composeThread]); + const fixedSearch = React.useMemo( + () => + searchStatus === 'active' ? renderSearch({ autoFocus: true }) : null, + [renderSearch, searchStatus], + ); + + const scrollEnabled = + searchStatus === 'inactive' || searchStatus === 'active'; + // viewerID is in extraData since it's used by MessagePreview + // within ChatThreadListItem + const viewerID = loggedInUserInfo?.id; + const extraData = `${viewerID || ''} ${openedSwipeableID}`; + + const flatListRef = React.useRef(); + const chatThreadList = React.useMemo( + () => ( + + {fixedSearch} + + {floatingAction} + + ), + [ + extraData, + fixedSearch, + floatingAction, + indicatorStyle, + onEndReached, + onScroll, + partialListData, + renderItem, + scrollEnabled, + styles.container, + styles.flatList, + ], + ); + return ( ); }