refactor(use-long-press): Remove unnecessary window check in useEffect

minwork · Jul 10, 2024 · 25780ea · 25780ea
1 parent 394fdde
commit 25780ea
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/packages/use-long-press/src/lib/use-long-press.ts b/packages/use-long-press/src/lib/use-long-press.ts
@@ -301,9 +301,6 @@ export function useLongPress<
 
   // Listen to long press stop events on window
   useEffect(() => {
-    // Do nothing if SSR
-    if (!window) return;
-
     function listener(event: LongPressDomEvents) {
       const reactEvent = createArtificialReactEvent<Target>(event);
       cancel()(reactEvent);

diff --git a/packages/use-long-press/src/tests/use-long-press.test.tsx b/packages/use-long-press/src/tests/use-long-press.test.tsx
@@ -201,7 +201,7 @@ describe('Different environment compatibility', () => {
     beforeEach(() => {
       // Use fake timers for detecting long press
       vi.useFakeTimers();
-      // Simulate absence of TouchEvent
+      // Simulate absence of window
       vi.stubGlobal('window', undefined);
     });