Merge pull request #49970 from bernhardoj/fix/49537-holding-down-item…

…-list-scrolls-the-list

Fix holding on a message in search page scrolls the page

src/components/SelectionList/BaseSelectionList.tsx

@@ -101,6 +101,7 @@ function BaseSelectionList<TItem extends ListItem>(
         onLongPressRow,
         shouldShowTextInput = !!textInputLabel || !!textInputIconLeft,
         shouldShowListEmptyContent = true,
+        shouldIgnoreFocus = false,
         scrollEventThrottle,
         contentContainerStyle,
     }: BaseSelectionListProps<TItem>,
@@ -468,7 +469,7 @@ function BaseSelectionList<TItem extends ListItem>(
                     isAlternateTextMultilineSupported={isAlternateTextMultilineSupported}
                     alternateTextNumberOfLines={alternateTextNumberOfLines}
                     onFocus={() => {
-                        if (isDisabled) {
+                        if (shouldIgnoreFocus || isDisabled) {
                             return;
                         }
                         setFocusedIndex(normalizedIndex);

src/components/SelectionList/index.tsx

@@ -1,6 +1,7 @@
 import React, {forwardRef, useEffect, useState} from 'react';
 import type {ForwardedRef} from 'react';
 import {Keyboard} from 'react-native';
+import * as Browser from '@libs/Browser';
 import * as DeviceCapabilities from '@libs/DeviceCapabilities';
 import BaseSelectionList from './BaseSelectionList';
 import type {BaseSelectionListProps, ListItem, SelectionListHandle} from './types';
@@ -42,6 +43,9 @@ function SelectionList<TItem extends ListItem>({onScroll, ...props}: BaseSelecti
             {...props}
             ref={ref}
             onScroll={onScroll ?? defaultOnScroll}
+            // Ignore the focus if it's caused by a touch event on mobile chrome.
+            // For example, a long press will trigger a focus event on mobile chrome.
+            shouldIgnoreFocus={Browser.isMobileChrome() && isScreenTouched}
         />
     );
 }

src/components/SelectionList/types.ts

@@ -492,6 +492,9 @@ type BaseSelectionListProps<TItem extends ListItem> = Partial<ChildrenProps> & {
     /** Styles to apply to SectionList component */
     sectionListStyle?: StyleProp<ViewStyle>;
 
+    /** Whether to ignore the focus event */
+    shouldIgnoreFocus?: boolean;
+
     /** Whether focus event should be delayed */
     shouldDelayFocus?: boolean;