@trackunit/react-drawer 2.9.5 → 2.10.1

package/index.cjs.js

@@ -526,7 +526,7 @@ const useDrawer = (props) => {
     // `false` either when it transitions `false -> true` again or once Drawer's
     // `onExitComplete` signals the panel has genuinely finished exiting
     // (`handleExitComplete`). See `UseDrawerReturnValue.isExiting`.
-    const [isExiting, setIsExiting] = react.useState(false);
+    const [committedIsExiting, setCommittedIsExiting] = react.useState(false);
     // Tracks the resolved `isOpen` value as of the last render for which
     // `isExiting` was derived below, so the render-phase check can detect a
     // transition without an effect.
@@ -559,10 +559,15 @@ const useDrawer = (props) => {
     // to `false` and `prevIsOpen` has already caught up to it (from the render
     // that flipped `isExiting` to `true` in the first place), so `isOpen ===
     // prevIsOpen` and this block is a no-op on the next render either way.
-    if (isOpen !== prevIsOpen) {
+    const isOpenChanged = isOpen !== prevIsOpen;
+    if (isOpenChanged) {
         setPrevIsOpen(isOpen);
-        setIsExiting(!isOpen); // true when transitioning to closed, false when transitioning to open
+        setCommittedIsExiting(!isOpen); // true when transitioning to closed, false when transitioning to open
     }
+    // React still finishes the render that called `setCommittedIsExiting` above before discarding it, and
+    // side effects in that pass (e.g. an Apollo query's `skip` read from `useHold`) are not undone.
+    // Returning the new value in that pass too keeps `isExiting` identical in both passes.
+    const isExiting = isOpenChanged ? !isOpen : committedIsExiting;
     const isPendingCloseRef = react.useRef(false);
     const onCloseRef = react.useRef(onClose);
     const onOpenRef = react.useRef(onOpen);
@@ -582,7 +587,7 @@ const useDrawer = (props) => {
     // Fired by `Drawer`'s internal `onExitComplete` prop exactly once per genuine
     // close→fully-exited cycle — the normal path back to `isExiting: false`.
     const handleExitComplete = react.useCallback(() => {
-        setIsExiting(false);
+        setCommittedIsExiting(false);
     }, []);
     const requestClose = react.useCallback((event, reason) => {
         if (onBeforeCloseRef.current) {

package/index.esm.js

@@ -524,7 +524,7 @@ const useDrawer = (props) => {
     // `false` either when it transitions `false -> true` again or once Drawer's
     // `onExitComplete` signals the panel has genuinely finished exiting
     // (`handleExitComplete`). See `UseDrawerReturnValue.isExiting`.
-    const [isExiting, setIsExiting] = useState(false);
+    const [committedIsExiting, setCommittedIsExiting] = useState(false);
     // Tracks the resolved `isOpen` value as of the last render for which
     // `isExiting` was derived below, so the render-phase check can detect a
     // transition without an effect.
@@ -557,10 +557,15 @@ const useDrawer = (props) => {
     // to `false` and `prevIsOpen` has already caught up to it (from the render
     // that flipped `isExiting` to `true` in the first place), so `isOpen ===
     // prevIsOpen` and this block is a no-op on the next render either way.
-    if (isOpen !== prevIsOpen) {
+    const isOpenChanged = isOpen !== prevIsOpen;
+    if (isOpenChanged) {
         setPrevIsOpen(isOpen);
-        setIsExiting(!isOpen); // true when transitioning to closed, false when transitioning to open
+        setCommittedIsExiting(!isOpen); // true when transitioning to closed, false when transitioning to open
     }
+    // React still finishes the render that called `setCommittedIsExiting` above before discarding it, and
+    // side effects in that pass (e.g. an Apollo query's `skip` read from `useHold`) are not undone.
+    // Returning the new value in that pass too keeps `isExiting` identical in both passes.
+    const isExiting = isOpenChanged ? !isOpen : committedIsExiting;
     const isPendingCloseRef = useRef(false);
     const onCloseRef = useRef(onClose);
     const onOpenRef = useRef(onOpen);
@@ -580,7 +585,7 @@ const useDrawer = (props) => {
     // Fired by `Drawer`'s internal `onExitComplete` prop exactly once per genuine
     // close→fully-exited cycle — the normal path back to `isExiting: false`.
     const handleExitComplete = useCallback(() => {
-        setIsExiting(false);
+        setCommittedIsExiting(false);
     }, []);
     const requestClose = useCallback((event, reason) => {
         if (onBeforeCloseRef.current) {

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "@trackunit/react-drawer",
-  "version": "2.9.5",
+  "version": "2.10.1",
   "repository": "https://github.com/Trackunit/manager",
   "license": "SEE LICENSE IN LICENSE.txt",
   "engines": {
     "node": ">=24.x"
   },
   "dependencies": {
-    "@trackunit/react-components": "3.5.5",
-    "@trackunit/css-class-variance-utilities": "2.2.2",
-    "@trackunit/i18n-library-translation": "2.7.3",
+    "@trackunit/react-components": "3.7.0",
+    "@trackunit/css-class-variance-utilities": "2.3.0",
+    "@trackunit/i18n-library-translation": "2.8.0",
     "@floating-ui/react": "^0.26.25",
     "tailwind-merge": "^2.0.0"
   },

package/migrations/entry.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"entry.js","sourceRoot":"","sources":["../../../../../libs/react/drawer/migrations/entry.ts"],"names":[],"mappings":"","sourcesContent":["export {};\n"]}