react-resizable-panels 2.1.2 → 2.1.4

package/CHANGELOG.md

@@ -1,6 +1,16 @@
 # Changelog
 
+## 2.1.4
+
+- Improve TypeScript HTML tag type generics (#407)
+- Edge case check to make sure resize handle hasn't been unmounted while dragging (#410)
+
+## 2.1.3
+
+- Edge case bug fix for a resize handle unmounting while being dragged (#402)
+
 ## 2.1.2
+
 - Suppress invalid layout warning for empty panel groups (#396)
 
 ## 2.1.1

package/dist/declarations/src/Panel.d.ts

@@ -30,7 +30,7 @@ export type ImperativePanelHandle = {
     isExpanded: () => boolean;
     resize: (size: number) => void;
 };
-export type PanelProps = Omit<HTMLAttributes<keyof HTMLElementTagNameMap>, "id" | "onResize"> & PropsWithChildren<{
+export type PanelProps<T extends keyof HTMLElementTagNameMap = keyof HTMLElementTagNameMap> = Omit<HTMLAttributes<HTMLElementTagNameMap[T]>, "id" | "onResize"> & PropsWithChildren<{
     className?: string;
     collapsedSize?: number | undefined;
     collapsible?: boolean | undefined;
@@ -43,7 +43,7 @@ export type PanelProps = Omit<HTMLAttributes<keyof HTMLElementTagNameMap>, "id"
     onResize?: PanelOnResize;
     order?: number;
     style?: object;
-    tagName?: keyof HTMLElementTagNameMap;
+    tagName?: T;
 }>;
 export declare function PanelWithForwardedRef({ children, className: classNameFromProps, collapsedSize, collapsible, defaultSize, forwardedRef, id: idFromProps, maxSize, minSize, onCollapse, onExpand, onResize, order, style: styleFromProps, tagName: Type, ...rest }: PanelProps & {
     forwardedRef: ForwardedRef<ImperativePanelHandle>;
@@ -51,7 +51,7 @@ export declare function PanelWithForwardedRef({ children, className: classNameFr
 export declare namespace PanelWithForwardedRef {
     var displayName: string;
 }
-export declare const Panel: import("react").ForwardRefExoticComponent<Omit<HTMLAttributes<keyof HTMLElementTagNameMap>, "id" | "onResize"> & {
+export declare const Panel: import("react").ForwardRefExoticComponent<Omit<HTMLAttributes<HTMLObjectElement | HTMLElement | HTMLSlotElement | HTMLStyleElement | HTMLTitleElement | HTMLAnchorElement | HTMLAreaElement | HTMLAudioElement | HTMLBaseElement | HTMLQuoteElement | HTMLBodyElement | HTMLBRElement | HTMLButtonElement | HTMLCanvasElement | HTMLTableCaptionElement | HTMLTableColElement | HTMLDataElement | HTMLDataListElement | HTMLModElement | HTMLDetailsElement | HTMLDialogElement | HTMLDivElement | HTMLDListElement | HTMLEmbedElement | HTMLFieldSetElement | HTMLFormElement | HTMLHeadingElement | HTMLHeadElement | HTMLHRElement | HTMLHtmlElement | HTMLIFrameElement | HTMLImageElement | HTMLInputElement | HTMLLabelElement | HTMLLegendElement | HTMLLIElement | HTMLLinkElement | HTMLMapElement | HTMLMenuElement | HTMLMetaElement | HTMLMeterElement | HTMLOListElement | HTMLOptGroupElement | HTMLOptionElement | HTMLOutputElement | HTMLParagraphElement | HTMLPictureElement | HTMLPreElement | HTMLProgressElement | HTMLScriptElement | HTMLSelectElement | HTMLSourceElement | HTMLSpanElement | HTMLTableElement | HTMLTableSectionElement | HTMLTableCellElement | HTMLTemplateElement | HTMLTextAreaElement | HTMLTimeElement | HTMLTableRowElement | HTMLTrackElement | HTMLUListElement | HTMLVideoElement>, "id" | "onResize"> & {
     className?: string | undefined;
     collapsedSize?: number | undefined;
     collapsible?: boolean | undefined;

package/dist/declarations/src/PanelResizeHandleRegistry.d.ts

@@ -1,6 +1,6 @@
 import { Direction, ResizeEvent } from "./types.js";
 export type ResizeHandlerAction = "down" | "move" | "up";
-export type SetResizeHandlerState = (action: ResizeHandlerAction, isActive: boolean, event: ResizeEvent) => void;
+export type SetResizeHandlerState = (action: ResizeHandlerAction, isActive: boolean, event: ResizeEvent | null) => void;
 export type PointerHitAreaMargins = {
     coarse: number;
     fine: number;

package/dist/react-resizable-panels.browser.cjs.js

@@ -485,6 +485,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -596,7 +600,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2082,6 +2086,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2312,6 +2318,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2329,6 +2336,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.browser.development.cjs.js

@@ -491,6 +491,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -602,7 +606,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2188,6 +2192,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2418,6 +2424,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2435,6 +2442,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.browser.development.esm.js

@@ -467,6 +467,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -578,7 +582,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2164,6 +2168,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2394,6 +2400,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2411,6 +2418,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.browser.esm.js

@@ -461,6 +461,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -572,7 +576,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2058,6 +2062,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2288,6 +2294,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2305,6 +2312,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.cjs.js

@@ -487,6 +487,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -598,7 +602,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2084,6 +2088,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2314,6 +2320,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2331,6 +2338,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.development.cjs.js

@@ -498,6 +498,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -609,7 +613,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2195,6 +2199,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2425,6 +2431,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2442,6 +2449,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.development.esm.js

@@ -474,6 +474,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -585,7 +589,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2171,6 +2175,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2401,6 +2407,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2418,6 +2425,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.development.node.cjs.js

@@ -460,6 +460,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -571,7 +575,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -1963,6 +1967,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2190,6 +2196,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2207,6 +2214,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.development.node.esm.js

@@ -436,6 +436,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -547,7 +551,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -1939,6 +1943,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2166,6 +2172,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2183,6 +2190,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.esm.js

@@ -463,6 +463,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -574,7 +578,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -2060,6 +2064,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2290,6 +2296,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2307,6 +2314,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.node.cjs.js

@@ -449,6 +449,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -560,7 +564,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -1862,6 +1866,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2089,6 +2095,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2106,6 +2113,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/dist/react-resizable-panels.node.esm.js

@@ -425,6 +425,10 @@ function registerResizeHandle(resizeHandleId, element, direction, hitAreaMargins
         intersectingHandles.splice(index, 1);
       }
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -536,7 +540,7 @@ function recalculateIntersectingHandles({
       // Clicking on an element "above" a handle (e.g. a modal) should prevent a hit though
       // so at this point we need to compare stacking order of a potentially intersecting drag handle,
       // and the element that was actually clicked/touched
-      if (targetElement !== null && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
+      if (targetElement !== null && document.contains(targetElement) && dragHandleElement !== targetElement && !dragHandleElement.contains(targetElement) && !targetElement.contains(dragHandleElement) &&
       // Calculating stacking order has a cost, so we should avoid it if possible
       // That is why we only check potentially intersecting handles,
       // and why we skip if the event target is within the handle's DOM
@@ -1838,6 +1842,8 @@ function PanelGroupWithForwardedRef({
       resizePanel(panelData, nextMaxSize);
     }
   }, [resizePanel]);
+
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback((dragHandleId, event) => {
     const {
       direction
@@ -2065,6 +2071,7 @@ function PanelResizeHandle({
           case "down":
             {
               setState("drag");
+              assert(event, 'Expected event to be defined for "down" action');
               startDragging(resizeHandleId, event);
               const {
                 onDragging
@@ -2082,6 +2089,7 @@ function PanelResizeHandle({
               if (state !== "drag") {
                 setState("hover");
               }
+              assert(event, 'Expected event to be defined for "move" action');
               resizeHandler(event);
               break;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-resizable-panels",
-  "version": "2.1.2",
+  "version": "2.1.4",
   "description": "React components for resizable panel groups/layouts",
   "author": "Brian Vaughn <brian.david.vaughn@gmail.com>",
   "license": "MIT",

package/src/Panel.ts

@@ -54,10 +54,9 @@ export type ImperativePanelHandle = {
   resize: (size: number) => void;
 };
 
-export type PanelProps = Omit<
-  HTMLAttributes<keyof HTMLElementTagNameMap>,
-  "id" | "onResize"
-> &
+export type PanelProps<
+  T extends keyof HTMLElementTagNameMap = keyof HTMLElementTagNameMap,
+> = Omit<HTMLAttributes<HTMLElementTagNameMap[T]>, "id" | "onResize"> &
   PropsWithChildren<{
     className?: string;
     collapsedSize?: number | undefined;
@@ -71,7 +70,7 @@ export type PanelProps = Omit<
     onResize?: PanelOnResize;
     order?: number;
     style?: object;
-    tagName?: keyof HTMLElementTagNameMap;
+    tagName?: T;
   }>;
 
 export function PanelWithForwardedRef({

package/src/PanelGroup.ts

@@ -816,6 +816,7 @@ function PanelGroupWithForwardedRef({
     [resizePanel]
   );
 
+  // TODO Multiple drag handles can be active at the same time so this API is a bit awkward now
   const startDragging = useCallback(
     (dragHandleId: string, event: ResizeEvent) => {
       const { direction } = committedValuesRef.current;

package/src/PanelResizeHandle.ts

@@ -129,13 +129,15 @@ export function PanelResizeHandle({
     const setResizeHandlerState = (
       action: ResizeHandlerAction,
       isActive: boolean,
-      event: ResizeEvent
+      event: ResizeEvent | null
     ) => {
       if (isActive) {
         switch (action) {
           case "down": {
             setState("drag");
 
+            assert(event, 'Expected event to be defined for "down" action');
+
             startDragging(resizeHandleId, event);
 
             const { onDragging } = callbacksRef.current;
@@ -151,6 +153,8 @@ export function PanelResizeHandle({
               setState("hover");
             }
 
+            assert(event, 'Expected event to be defined for "move" action');
+
             resizeHandler(event);
             break;
           }

package/src/PanelResizeHandleRegistry.ts

@@ -9,7 +9,7 @@ export type ResizeHandlerAction = "down" | "move" | "up";
 export type SetResizeHandlerState = (
   action: ResizeHandlerAction,
   isActive: boolean,
-  event: ResizeEvent
+  event: ResizeEvent | null
 ) => void;
 
 export type PointerHitAreaMargins = {
@@ -83,6 +83,10 @@ export function registerResizeHandle(
       }
 
       updateCursor();
+
+      // Also instruct the handle to stop dragging; this prevents the parent group from being left in an inconsistent state
+      // See github.com/bvaughn/react-resizable-panels/issues/402
+      setResizeHandlerState("up", true, null);
     }
   };
 }
@@ -193,6 +197,7 @@ function recalculateIntersectingHandles({
       // and the element that was actually clicked/touched
       if (
         targetElement !== null &&
+        document.contains(targetElement) &&
         dragHandleElement !== targetElement &&
         !dragHandleElement.contains(targetElement) &&
         !targetElement.contains(dragHandleElement) &&