Fix returning to previous thread when clicking on the video player

Expensify · Feb 27, 2024 · 283981c · 283981c
1 parent 9bfe961
commit 283981c
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/components/VideoPlayer/BaseVideoPlayer.js b/src/components/VideoPlayer/BaseVideoPlayer.js
@@ -169,7 +169,12 @@ function BaseVideoPlayer({
 
     return (
         <>
-            <View style={style}>
+            {/* 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.cursorDefault, style]}
+            >
                 <Hoverable>
                     {(isHovered) => (
                         <View style={[styles.w100, styles.h100]}>
@@ -241,7 +246,7 @@ function BaseVideoPlayer({
                         </View>
                     )}
                 </Hoverable>
-            </View>
+            </PressableWithoutFeedback>
             <VideoPopoverMenu
                 isPopoverVisible={isPopoverVisible}
                 hidePopover={hidePopoverMenu}