feat: import utility functions from @tonic-ui/utils

trendmicro-frontend · Nov 15, 2023 · c8681c6 · c8681c6
1 parent 28da282
commit c8681c6
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 21 deletions.
diff --git a/packages/react-hooks/src/useEventListener.js b/packages/react-hooks/src/useEventListener.js
@@ -1,17 +1,7 @@
+import { noop, runIfFn } from '@tonic-ui/utils';
 import { useEffect } from 'react';
 import useEventCallback from './useEventCallback';
 
-// TODO: move to '@tonic-ui/utils'
-const noop = () => {};
-
-// TODO: move to '@tonic-ui/utils'
-const runIfFn = (valueOrFn, ...args) => {
-  if (typeof valueOrFn === 'function') {
-    return valueOrFn(...args);
-  }
-  return valueOrFn;
-};
-
 /**
  * A custom Hook to manage browser event listeners.
  *

diff --git a/packages/react-hooks/src/useMediaQuery.js b/packages/react-hooks/src/useMediaQuery.js
@@ -1,8 +1,6 @@
+import { noop } from '@tonic-ui/utils';
 import { useEffect, useState } from 'react';
 
-// TODO: move to '@tonic-ui/utils'
-const noop = () => {};
-
 const getInitialState = (query, defaultValue) => {
   if (defaultValue !== undefined) {
     return defaultValue;

diff --git a/packages/react-hooks/src/useOutsideClick.js b/packages/react-hooks/src/useOutsideClick.js
@@ -1,12 +1,7 @@
+import { getOwnerDocument, noop } from '@tonic-ui/utils';
 import { useEffect } from 'react';
 import useEventCallback from './useEventCallback';
 
-// TODO: move to '@tonic-ui/utils'
-const noop = () => {};
-
-// TODO: move to '@tonic-ui/utils'
-const ownerDocument = (node) => (node?.ownerDocument || document);
-
 const defaultEvents = ['mousedown', 'touchstart'];
 
 /**
@@ -33,7 +28,7 @@ const useOutsideClick = (
 
     const filteredEvents = (Array.isArray(events) ? events : [])
       .filter(x => (typeof x === 'string'));
-    const doc = ownerDocument(ref?.current);
+    const doc = getOwnerDocument(ref?.current);
 
     for (const eventName of filteredEvents) {
       doc?.addEventListener?.(eventName, savedHandler, true);