Merge pull request #37161 from Expensify/revert-37036-video-player-fo…

…llowups/app-returns-to-previous-thread-when-pressed-on-video [CP Staging] Revert "Fix app returns to previous thread when scrubbing video in thread"
Expensify · Feb 23, 2024 · 07681d4 · 07681d4
2 parents cad3114 + 55d92d5
commit 07681d4
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/components/VideoPlayer/BaseVideoPlayer.js b/src/components/VideoPlayer/BaseVideoPlayer.js
@@ -168,12 +168,7 @@ function BaseVideoPlayer({
     }, [bindFunctions, currentVideoPlayerRef, currentlyPlayingURL, isSmallScreenWidth, originalParent, sharedElement, shouldUseSharedVideoElement, url]);
 
     return (
-        // We need to wrap the video component in a component that will catch unhandled pointer events. Otherwise, these
-        // events will bubble up the tree, and it will cause unexpected press behavior.
-        <PressableWithoutFeedback
-            accessibilityRole="button"
-            style={styles.flex1}
-        >
+        <>
             <View style={style}>
                 <Hoverable>
                     {(isHovered) => (
@@ -252,7 +247,7 @@ function BaseVideoPlayer({
                 hidePopover={hidePopoverMenu}
                 anchorPosition={popoverAnchorPosition}
             />
-        </PressableWithoutFeedback>
+        </>
     );
 }