Fix forget to use the ref parameter (#1383)

voyagegroup · Aug 22, 2023 · e69e7c9 · e69e7c9
1 parent a17cfec
commit e69e7c9
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 10 deletions.
diff --git a/.changeset/chilly-rabbits-play.md b/.changeset/chilly-rabbits-play.md
@@ -0,0 +1,5 @@
+---
+"ingred-ui": patch
+---
+
+fix forget to use the ref parameter
diff --git a/src/components/NewDateRangePicker/NewDateRangePicker.tsx b/src/components/NewDateRangePicker/NewDateRangePicker.tsx
@@ -15,6 +15,7 @@ import {
   ClickState,
   ClickStateType,
 } from "../Calendar/CalendarRange/constants";
+import { useMergeRefs } from "../../hooks/useMergeRefs";
 
 export type DateRange = {
   startDate: Dayjs;
@@ -63,23 +64,28 @@ export type NewDateRangePickerProps = {
 export const DateRangePicker = forwardRef<
   HTMLDivElement,
   NewDateRangePickerProps
->(function DateRangePicker({
-  startDate,
-  endDate,
-  errorText,
-  disabled = false,
-  isOutsideRange = () => false,
-  actions,
-  onDatesChange,
-}) {
+>(function DateRangePicker(
+  {
+    startDate,
+    endDate,
+    errorText,
+    disabled = false,
+    isOutsideRange = () => false,
+    actions,
+    onDatesChange,
+  },
+  propRef,
+) {
   const [open, setOpen] = useState(false);
   const { context, refs, strategy, x, y } = useFloating({
     placement: "right-start",
     open,
     onOpenChange: setOpen,
     middleware: [offset(10), flip()],
   });
-  const ref = useRef<HTMLDivElement>(null);
+  const innerRef = useRef<HTMLDivElement>(null);
+  const ref = useMergeRefs(propRef, innerRef);
+
   const dismiss = useDismiss(context);
   const role = useRole(context);