From 75dfd28eec430cde4156ebf8b25574e9bdb48257 Mon Sep 17 00:00:00 2001
From: Navin Moorthy <navin007.a@gmail.com>
Date: Thu, 16 Jun 2022 15:45:43 +0530
Subject: [PATCH] =?UTF-8?q?fix(disclosure):=20=F0=9F=90=9B=20remove=20call?=
 =?UTF-8?q?backs=20being=20passed=20to=20the=20dom?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../disclosure-collapsible-content.ts           | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/disclosure/disclosure-collapsible-content.ts b/src/disclosure/disclosure-collapsible-content.ts
index cc7b73d65..732ce2c63 100644
--- a/src/disclosure/disclosure-collapsible-content.ts
+++ b/src/disclosure/disclosure-collapsible-content.ts
@@ -36,10 +36,15 @@ import {
 export const useDisclosureCollapsibleContent =
   createHook<DisclosureCollapsibleContentOptions>(
     ({
+      state,
       direction = "vertical",
       contentSize = 0,
       easing = "cubic-bezier(0.4, 0, 0.2, 1)",
-      state,
+      duration,
+      onExpandStart,
+      onExpandEnd,
+      onCollapseStart,
+      onCollapseEnd,
       ...props
     }) => {
       const id = useId(props.id);
@@ -78,7 +83,7 @@ export const useDisclosureCollapsibleContent =
       function getTransitionStyles(size: number | string): {
         transition?: string;
       } {
-        const _duration = props.duration || getAutoSizeDuration(size);
+        const _duration = duration || getAutoSizeDuration(size);
 
         return {
           transition: `${currentSize} ${_duration}ms ${easing}`,
@@ -88,7 +93,7 @@ export const useDisclosureCollapsibleContent =
       useUpdateEffect(() => {
         if (state.visible) {
           raf(() => {
-            props.onExpandStart?.();
+            onExpandStart?.();
 
             mergeStyles({
               willChange: `${currentSize}`,
@@ -108,7 +113,7 @@ export const useDisclosureCollapsibleContent =
           });
         } else {
           raf(() => {
-            props.onCollapseStart?.();
+            onCollapseStart?.();
 
             const size = isVertical
               ? getElementHeight(contentRef)
@@ -171,14 +176,14 @@ export const useDisclosureCollapsibleContent =
               });
             }
 
-            props.onExpandEnd?.();
+            onExpandEnd?.();
 
             // If the height we should be animating to matches the collapsed height,
             // it's safe to apply the collapsed overrides
           } else if (stylesSize === `${contentSize}px`) {
             setStyles(collapsedStyles);
 
-            props.onCollapseEnd?.();
+            onCollapseEnd?.();
           }
         },
       );