From 4ca4e7472220e8c4defb64a6e034f0ac6cbc26a1 Mon Sep 17 00:00:00 2001 From: Chirag Chandrakant Salian Date: Fri, 23 Feb 2024 12:46:22 -0800 Subject: [PATCH] Merge pull request #37161 from Expensify/revert-37036-video-player-followups/app-returns-to-previous-thread-when-pressed-on-video [CP Staging] Revert "Fix app returns to previous thread when scrubbing video in thread" (cherry picked from commit 07681d441e2eab658b0951c54c1238085bfafb97) --- src/components/VideoPlayer/BaseVideoPlayer.js | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/components/VideoPlayer/BaseVideoPlayer.js b/src/components/VideoPlayer/BaseVideoPlayer.js index 5f63703b462f..df79c7ef18da 100644 --- 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. - + <> {(isHovered) => ( @@ -252,7 +247,7 @@ function BaseVideoPlayer({ hidePopover={hidePopoverMenu} anchorPosition={popoverAnchorPosition} /> - + ); }