fix: show / hide default video controls on mobile

pmndrs · Jun 25, 2024 · c6f2e3f · c6f2e3f
1 parent d949959
commit c6f2e3f
Showing 1 changed file with 7 additions and 14 deletions.
diff --git a/packages/kits/default/src/video.tsx b/packages/kits/default/src/video.tsx
@@ -38,23 +38,16 @@ export const Video: (props: VideoProperties & RefAttributes<VideoInternals>) =>
     const moving = useMemo(() => signal(false), [])
     const handlers = useMemo<EventHandlers>(() => {
       let timeoutRef: NodeJS.Timeout | undefined
-      const cancelTimeout = () => {
-        if (timeoutRef == null) {
-          return
+      const onInteract = () => {
+        moving.value = true
+        if (timeoutRef != null) {
+          clearTimeout(timeoutRef)
         }
-        clearTimeout(timeoutRef)
-        timeoutRef = undefined
+        timeoutRef = setTimeout(() => (moving.value = false), 2000)
       }
       return {
-        onPointerMove: () => {
-          moving.value = true
-          cancelTimeout()
-          timeoutRef = setTimeout(() => (moving.value = false), 2000)
-        },
-        onPointerOut: () => {
-          moving.value = false
-          cancelTimeout
-        },
+        onPointerMove: onInteract,
+        onPointerDown: onInteract,
       }
     }, [moving])
     return (