fix(Disclosure): content visibility style (#908)

Author: tenphi

.changeset/tasty-drinks-invent.md

@@ -0,0 +1,5 @@
+---
+"@cube-dev/ui-kit": patch
+---
+
+Fix content visibility in Disclosure.

src/components/actions/ItemButton/ItemButton.tsx

@@ -100,6 +100,12 @@ const ItemButton = forwardRef(function ItemButton(
     type = 'neutral',
     theme,
     onPress,
+    // Extract react-aria press callbacks to prevent them from leaking to DOM via rest.
+    // These are handled by useButton inside useAction.
+    onPressStart: _onPressStart,
+    onPressEnd: _onPressEnd,
+    onPressChange: _onPressChange,
+    onPressUp: _onPressUp,
     actions,
     size = 'medium',
     wrapperStyles,

src/components/content/Disclosure/Disclosure.tsx

@@ -80,7 +80,7 @@ export interface CubeDisclosureProps
 }
 
 export interface CubeDisclosureTriggerProps
-  extends Omit<CubeItemButtonProps, 'onPress'> {
+  extends Omit<CubeItemButtonProps, 'onPress' | 'isDisabled'> {
   /** Children content for the trigger */
   children?: ReactNode;
 }
@@ -184,13 +184,14 @@ const ContentWrapperElement = tasty({
       '': '0',
       shown: 'max-content',
     },
-    transition: 'height $disclosure-transition linear',
+    transition: 'height $disclosure-transition',
   },
 });
 
 const ContentElement = tasty({
   qa: 'DisclosureContent',
   styles: {
+    contentVisibility: 'auto',
     padding: '1x',
   },
 });

src/components/helpers/DisplayTransition/DisplayTransition.tsx

@@ -327,10 +327,8 @@ export function DisplayTransition({
   const refCallback: RefCallback<HTMLElement> = (node) => {
     if (node) {
       elementRef.current = node;
-
-      if (phaseRef.current === 'enter') {
-        ensureEnterFlow();
-      }
+      // Don't call ensureEnterFlow() here - useLayoutEffect handles RAF scheduling
+      // to ensure symmetric timing with exit flow
     } else {
       cleanupEventListeners();
       elementRef.current = null;