Update useMergedRefs

actualbudget · Apr 11, 2024 · e0dd586 · e0dd586
1 parent a1e7e6c
commit e0dd586
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 4 deletions.
diff --git a/packages/desktop-client/src/components/mobile/transactions/FocusableAmountInput.tsx b/packages/desktop-client/src/components/mobile/transactions/FocusableAmountInput.tsx
@@ -47,10 +47,9 @@ const AmountInput = memo(function AmountInput({
   const inputRef = useRef<HTMLInputElement>();
   const [hideFraction = false] = useLocalPref('hideFraction');
 
-  // Need to update useMergedRefs to support Ref.
   const mergedInputRef = useMergedRefs<HTMLInputElement>(
-    props.inputRef as MutableRefObject<HTMLInputElement>,
-    inputRef as MutableRefObject<HTMLInputElement>,
+    props.inputRef,
+    inputRef,
   );
 
   const initialValue = Math.abs(props.value);

diff --git a/packages/desktop-client/src/hooks/useMergedRefs.ts b/packages/desktop-client/src/hooks/useMergedRefs.ts
@@ -2,7 +2,13 @@ import { useCallback } from 'react';
 import type { MutableRefObject, Ref, RefCallback } from 'react';
 
 export function useMergedRefs<T>(
-  ...refs: (RefCallback<T> | MutableRefObject<T> | Ref<T> | null | undefined)[]
+  ...refs: (
+    | RefCallback<T | null | undefined>
+    | MutableRefObject<T | null | undefined>
+    | Ref<T | null | undefined>
+    | null
+    | undefined
+  )[]
 ): Ref<T> {
   return useCallback(
     (value: T) => {