fix: update and forceUpdate TS types

floating-ui · Apr 11, 2020 · 5f7bca5 · 5f7bca5
1 parent 8a034c6
commit 5f7bca5
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 6 deletions.
diff --git a/typings/react-popper.d.ts b/typings/react-popper.d.ts
@@ -58,6 +58,6 @@ export function usePopper(
   styles: { [key: string]: React.CSSProperties };
   attributes: { [key: string]: { [key: string]: string } };
   state: PopperJS.State;
-  update: Pick<PopperJS.Instance, 'update'>;
-  forceUpdate: Pick<PopperJS.Instance, 'forceUpdate'>;
+  update: PopperJS.Instance['update'] | null;
+  forceUpdate: PopperJS.Instance['forceUpdate'] | null;
 };
diff --git a/typings/tests/main-test.tsx b/typings/tests/main-test.tsx
@@ -56,13 +56,23 @@ const HookTest = () => {
   const [arrowElement, setArrowElement] = React.useState<HTMLElement | null>(
     null
   );
-  const { styles, attributes } = usePopper(referenceElement, popperElement, {
-    modifiers: [{ name: 'arrow', options: { element: arrowElement } }],
-  });
+  const { styles, attributes, update } = usePopper(
+    referenceElement,
+    popperElement,
+    {
+      modifiers: [{ name: 'arrow', options: { element: arrowElement } }],
+    }
+  );
 
   return (
     <>
-      <button type="button" ref={setReferenceElement}>
+      <button
+        type="button"
+        ref={setReferenceElement}
+        onClick={() => {
+          update && update();
+        }}
+      >
         Reference element
       </button>