dockview-core 7.0.2 → 7.0.4

package/dist/package/main.esm.mjs

@@ -1,6 +1,6 @@
 /**
  * dockview-core
- * @version 7.0.2
+ * @version 7.0.4
  * @link https://github.com/mathuo/dockview
  * @license MIT
  */
@@ -186,7 +186,7 @@ class LeakageMonitor {
 class Stacktrace {
     static create() {
         var _a;
-        return new Stacktrace((_a = new Error().stack) !== null && _a !== void 0 ? _a : '');
+        return new Stacktrace((_a = new Error('listener stacktrace').stack) !== null && _a !== void 0 ? _a : '');
     }
     constructor(value) {
         this.value = value;
@@ -435,7 +435,7 @@ class FocusTracker extends CompositeDisposable {
         const onBlur = () => {
             if (hasFocus) {
                 loosingFocus = true;
-                window.setTimeout(() => {
+                globalThis.setTimeout(() => {
                     if (loosingFocus) {
                         loosingFocus = false;
                         hasFocus = false;
@@ -813,7 +813,7 @@ class ViewItem {
         return this._cachedVisibleSize;
     }
     get visible() {
-        return typeof this._cachedVisibleSize === 'undefined';
+        return this._cachedVisibleSize === undefined;
     }
     get minimumSize() {
         return this.visible ? this.view.minimumSize : 0;
@@ -1215,7 +1215,7 @@ class Splitview {
         const viewItem = new ViewItem(container, view, viewSize, {
             dispose: () => {
                 disposable.dispose();
-                this.viewContainer.removeChild(container);
+                container.remove();
             },
         });
         if (index === this.viewItems.length) {
@@ -1291,29 +1291,30 @@ class Splitview {
                     this.distributeEmptySpace();
                     this.layoutViews();
                 };
+                const doc = sash.ownerDocument;
                 const end = () => {
                     for (const item of this.viewItems) {
                         item.enabled = true;
                     }
                     iframes.release();
                     this.saveProportions();
-                    document.removeEventListener('pointermove', onPointerMove);
-                    document.removeEventListener('pointerup', end);
-                    document.removeEventListener('pointercancel', end);
-                    document.removeEventListener('contextmenu', end);
+                    doc.removeEventListener('pointermove', onPointerMove);
+                    doc.removeEventListener('pointerup', end);
+                    doc.removeEventListener('pointercancel', end);
+                    doc.removeEventListener('contextmenu', end);
                     this._onDidSashEnd.fire(undefined);
                 };
-                document.addEventListener('pointermove', onPointerMove);
-                document.addEventListener('pointerup', end);
-                document.addEventListener('pointercancel', end);
-                document.addEventListener('contextmenu', end);
+                doc.addEventListener('pointermove', onPointerMove);
+                doc.addEventListener('pointerup', end);
+                doc.addEventListener('pointercancel', end);
+                doc.addEventListener('contextmenu', end);
             };
             sash.addEventListener('pointerdown', onPointerStart);
             const sashItem = {
                 container: sash,
                 disposable: () => {
                     sash.removeEventListener('pointerdown', onPointerStart);
-                    this.sashContainer.removeChild(sash);
+                    sash.remove();
                 },
             };
             this.sashContainer.appendChild(sash);
@@ -1360,7 +1361,7 @@ class Splitview {
         if (!skipLayout) {
             this.relayout();
         }
-        if (sizing && sizing.type === 'distribute') {
+        if ((sizing === null || sizing === void 0 ? void 0 : sizing.type) === 'distribute') {
             this.distributeViewSizes();
         }
         this._onDidRemoveView.fire(viewItem.view);
@@ -1375,7 +1376,7 @@ class Splitview {
     }
     moveView(from, to) {
         const cachedVisibleSize = this.getViewCachedVisibleSize(from);
-        const sizing = typeof cachedVisibleSize === 'undefined'
+        const sizing = cachedVisibleSize === undefined
             ? this.getViewSize(from)
             : Sizing.Invisible(cachedVisibleSize);
         const view = this.removeView(from, undefined, true);
@@ -1385,13 +1386,7 @@ class Splitview {
         const previousSize = Math.max(this.size, this._contentSize);
         this.size = size;
         this.orthogonalSize = orthogonalSize;
-        if (!this.proportions) {
-            const indexes = range(this.viewItems.length);
-            const lowPriorityIndexes = indexes.filter((i) => this.viewItems[i].priority === LayoutPriority.Low);
-            const highPriorityIndexes = indexes.filter((i) => this.viewItems[i].priority === LayoutPriority.High);
-            this.resize(this.viewItems.length - 1, size - previousSize, undefined, lowPriorityIndexes, highPriorityIndexes);
-        }
-        else {
+        if (this.proportions) {
             let total = 0;
             for (let i = 0; i < this.viewItems.length; i++) {
                 const item = this.viewItems[i];
@@ -1411,6 +1406,12 @@ class Splitview {
                 }
             }
         }
+        else {
+            const indexes = range(this.viewItems.length);
+            const lowPriorityIndexes = indexes.filter((i) => this.viewItems[i].priority === LayoutPriority.Low);
+            const highPriorityIndexes = indexes.filter((i) => this.viewItems[i].priority === LayoutPriority.High);
+            this.resize(this.viewItems.length - 1, size - previousSize, undefined, lowPriorityIndexes, highPriorityIndexes);
+        }
         this.distributeEmptySpace();
         this.layoutViews();
     }
@@ -1630,7 +1631,7 @@ class Splitview {
         this._onDidRemoveView.dispose();
         for (let i = 0; i < this.element.children.length; i++) {
             if (this.element.children.item(i) === this.element) {
-                this.element.removeChild(this.element);
+                this.element.remove();
                 break;
             }
         }
@@ -1967,13 +1968,11 @@ class BranchNode extends CompositeDisposable {
         if (this.children.length === 0) {
             return LayoutPriority.Normal;
         }
-        const priorities = this.children.map((c) => typeof c.priority === 'undefined'
-            ? LayoutPriority.Normal
-            : c.priority);
-        if (priorities.some((p) => p === LayoutPriority.High)) {
+        const priorities = new Set(this.children.map((c) => { var _a; return (_a = c.priority) !== null && _a !== void 0 ? _a : LayoutPriority.Normal; }));
+        if (priorities.has(LayoutPriority.High)) {
             return LayoutPriority.High;
         }
-        else if (priorities.some((p) => p === LayoutPriority.Low)) {
+        else if (priorities.has(LayoutPriority.Low)) {
             return LayoutPriority.Low;
         }
         return LayoutPriority.Normal;
@@ -2010,16 +2009,7 @@ class BranchNode extends CompositeDisposable {
         this._size = size;
         this.element = document.createElement('div');
         this.element.className = 'dv-branch-node';
-        if (!childDescriptors) {
-            this.splitview = new Splitview(this.element, {
-                orientation: this.orientation,
-                proportionalLayout,
-                styles,
-                margin,
-            });
-            this.splitview.layout(this.size, this.orthogonalSize);
-        }
-        else {
+        if (childDescriptors) {
             const descriptor = {
                 views: childDescriptors.map((childDescriptor) => {
                     return {
@@ -2042,6 +2032,15 @@ class BranchNode extends CompositeDisposable {
                 margin,
             });
         }
+        else {
+            this.splitview = new Splitview(this.element, {
+                orientation: this.orientation,
+                proportionalLayout,
+                styles,
+                margin,
+            });
+            this.splitview.layout(this.size, this.orthogonalSize);
+        }
         this.disabled = disabled;
         this.addDisposables(this._onDidChange, this._onDidVisibilityChange, this.splitview.onDidSashEnd(() => {
             this._onDidChange.fire({});
@@ -2539,7 +2538,7 @@ class Gridview {
         if (oldRoot) {
             oldRoot.dispose();
             this._maximizedNode = undefined;
-            this.element.removeChild(oldRoot.element);
+            oldRoot.element.remove();
         }
         this._root = root;
         this.element.appendChild(this._root.element);
@@ -2631,7 +2630,7 @@ class Gridview {
     progmaticSelect(location, reverse = false) {
         const [path, node] = this.getNode(location);
         if (!(node instanceof LeafNode)) {
-            throw new Error('invalid location');
+            throw new TypeError('invalid location');
         }
         for (let i = path.length - 1; i > -1; i--) {
             const n = path[i];
@@ -2668,7 +2667,7 @@ class Gridview {
         const [rest, index] = tail(location);
         const [, parent] = this.getNode(rest);
         if (!(parent instanceof BranchNode)) {
-            throw new Error('Invalid from location');
+            throw new TypeError('Invalid from location');
         }
         return parent.isChildVisible(index);
     }
@@ -2679,7 +2678,7 @@ class Gridview {
         const [rest, index] = tail(location);
         const [, parent] = this.getNode(rest);
         if (!(parent instanceof BranchNode)) {
-            throw new Error('Invalid from location');
+            throw new TypeError('Invalid from location');
         }
         this._onDidViewVisibilityChange.fire();
         parent.setChildVisible(index, visible);
@@ -2690,7 +2689,7 @@ class Gridview {
         }
         const [, parent] = this.getNode(parentLocation);
         if (!(parent instanceof BranchNode)) {
-            throw new Error('Invalid location');
+            throw new TypeError('Invalid location');
         }
         parent.moveChild(from, to);
     }
@@ -2736,11 +2735,11 @@ class Gridview {
         const [rest, index] = tail(location);
         const [pathToParent, parent] = this.getNode(rest);
         if (!(parent instanceof BranchNode)) {
-            throw new Error('Invalid location');
+            throw new TypeError('Invalid location');
         }
         const nodeToRemove = parent.children[index];
         if (!(nodeToRemove instanceof LeafNode)) {
-            throw new Error('Invalid location');
+            throw new TypeError('Invalid location');
         }
         parent.removeChild(index, sizing);
         nodeToRemove.dispose();
@@ -2822,7 +2821,7 @@ class Gridview {
             return [path, node];
         }
         if (!(node instanceof BranchNode)) {
-            throw new Error('Invalid location');
+            throw new TypeError('Invalid location');
         }
         const [index, ...rest] = location;
         if (index < 0 || index >= node.children.length) {
@@ -3002,8 +3001,7 @@ class BaseGrid extends Resizable {
         }), this.gridview.onDidViewVisibilityChange(() => this._onDidViewVisibilityChangeMicroTaskQueue.fire()), this.onDidViewVisibilityChangeMicroTaskQueue(() => {
             this.forceRelayout();
         }), Disposable.from(() => {
-            var _a;
-            (_a = this.element.parentElement) === null || _a === void 0 ? void 0 : _a.removeChild(this.element);
+            this.element.remove();
         }), this.gridview.onDidChange(() => {
             this._bufferOnDidLayoutChange.fire();
         }), Event.any(this.onDidAdd, this.onDidRemove, this.onDidActiveChange)(() => {
@@ -3093,9 +3091,7 @@ class BaseGrid extends Resizable {
     }
     moveToNext(options) {
         var _a;
-        if (!options) {
-            options = {};
-        }
+        options !== null && options !== void 0 ? options : (options = {});
         if (!options.group) {
             if (!this.activeGroup) {
                 return;
@@ -3108,9 +3104,7 @@ class BaseGrid extends Resizable {
     }
     moveToPrevious(options) {
         var _a;
-        if (!options) {
-            options = {};
-        }
+        options !== null && options !== void 0 ? options : (options = {});
         if (!options.group) {
             if (!this.activeGroup) {
                 return;
@@ -4612,9 +4606,8 @@ function addGhostImage(dataTransfer, ghostElement, options) {
  */
 class PointerDragController extends CompositeDisposable {
     static getInstance() {
-        if (!PointerDragController._instance) {
-            PointerDragController._instance = new PointerDragController();
-        }
+        var _a;
+        (_a = PointerDragController._instance) !== null && _a !== void 0 ? _a : (PointerDragController._instance = new PointerDragController());
         return PointerDragController._instance;
     }
     constructor() {
@@ -4679,21 +4672,24 @@ class PointerDragController extends CompositeDisposable {
         this._onDragStart.fire(startEvent);
         // Source's owning window — popout drags fire on their own window,
         // not the main one.
-        const targetWindow = (_c = (_b = source.ownerDocument) === null || _b === void 0 ? void 0 : _b.defaultView) !== null && _c !== void 0 ? _c : window;
+        const targetWindow = (_c = (_b = source.ownerDocument) === null || _b === void 0 ? void 0 : _b.defaultView) !== null && _c !== void 0 ? _c : globalThis.window;
         this._moveListener = addDisposableListener(targetWindow, 'pointermove', (e) => {
-            if (!this._active || e.pointerId !== this._active.pointerId) {
+            var _a;
+            if (e.pointerId !== ((_a = this._active) === null || _a === void 0 ? void 0 : _a.pointerId)) {
                 return;
             }
             this._handleMove(e);
         });
         this._upListener = addDisposableListener(targetWindow, 'pointerup', (e) => {
-            if (!this._active || e.pointerId !== this._active.pointerId) {
+            var _a;
+            if (e.pointerId !== ((_a = this._active) === null || _a === void 0 ? void 0 : _a.pointerId)) {
                 return;
             }
             this._handleEnd(e, true);
         });
         this._cancelListener = addDisposableListener(targetWindow, 'pointercancel', (e) => {
-            if (!this._active || e.pointerId !== this._active.pointerId) {
+            var _a;
+            if (e.pointerId !== ((_a = this._active) === null || _a === void 0 ? void 0 : _a.pointerId)) {
                 return;
             }
             this._handleEnd(e, false);
@@ -5033,7 +5029,9 @@ class PointerDragSource extends CompositeDisposable {
             ? initiationDelayOpt()
             : initiationDelayOpt)) !== null && _a !== void 0 ? _a : DEFAULT_TOUCH_INITIATION_DELAY;
         this._armed = !isTouch || initiationDelay <= 0;
-        if (isTouch && initiationDelay > 0 && isFinite(initiationDelay)) {
+        if (isTouch &&
+            initiationDelay > 0 &&
+            Number.isFinite(initiationDelay)) {
             this._armTimer = setTimeout(() => {
                 this._armTimer = undefined;
                 this._armed = true;
@@ -5045,7 +5043,7 @@ class PointerDragSource extends CompositeDisposable {
             ? pressToleranceOpt()
             : pressToleranceOpt)) !== null && _c !== void 0 ? _c : DEFAULT_PRESS_TOLERANCE;
         // Source's owning window — popout drags fire on their own window.
-        const targetWindow = (_e = (_d = this.element.ownerDocument) === null || _d === void 0 ? void 0 : _d.defaultView) !== null && _e !== void 0 ? _e : window;
+        const targetWindow = (_e = (_d = this.element.ownerDocument) === null || _d === void 0 ? void 0 : _d.defaultView) !== null && _e !== void 0 ? _e : globalThis.window;
         this._pendingMoveListener = addDisposableListener(targetWindow, 'pointermove', (moveEvent) => {
             if (moveEvent.pointerId !== this._pendingPointerId) {
                 return;
@@ -5808,7 +5806,7 @@ class DraggablePaneviewPanel extends PaneviewPanel {
     }
     onDrop(event) {
         const data = getPaneData();
-        if (!data || data.viewId !== this.accessor.id) {
+        if ((data === null || data === void 0 ? void 0 : data.viewId) !== this.accessor.id) {
             // if there is no local drag event for this panel
             // or if the drag event was creating by another Paneview instance
             this._onDidDrop.fire(Object.assign(Object.assign({}, event), { panel: this, api: new PaneviewApi(this.accessor), getData: getPaneData }));
@@ -5881,7 +5879,7 @@ class ContentContainer extends CompositeDisposable {
                 this.group.location.type !== 'floating') {
                 return false;
             }
-            if (data && data.viewId === this.accessor.id) {
+            if ((data === null || data === void 0 ? void 0 : data.viewId) === this.accessor.id) {
                 return true;
             }
             return this.group.canDisplayOverlay(event, position, 'content');
@@ -5948,7 +5946,7 @@ class ContentContainer extends CompositeDisposable {
             /**
              * If the currently attached panel is mounted directly to the content then remove it
              */
-            this._element.removeChild(this.panel.view.content.element);
+            this.panel.view.content.element.remove();
             (_b = (_a = this.panel.view.content).onHide) === null || _b === void 0 ? void 0 : _b.call(_a);
         }
         this.panel = panel;
@@ -5966,7 +5964,7 @@ class ContentContainer extends CompositeDisposable {
                 break;
             case 'always':
                 if (panel.view.content.element.parentElement === this._element) {
-                    this._element.removeChild(panel.view.content.element);
+                    panel.view.content.element.remove();
                 }
                 container = this.group.renderContainer.attach({
                     panel,
@@ -5994,11 +5992,11 @@ class ContentContainer extends CompositeDisposable {
         // noop
     }
     closePanel() {
-        var _a, _b, _c;
+        var _a, _b;
         if (this.panel) {
             if (this.panel.api.renderer === 'onlyWhenVisible') {
-                (_a = this.panel.view.content.element.parentElement) === null || _a === void 0 ? void 0 : _a.removeChild(this.panel.view.content.element);
-                (_c = (_b = this.panel.view.content).onHide) === null || _c === void 0 ? void 0 : _c.call(_b);
+                this.panel.view.content.element.remove();
+                (_b = (_a = this.panel.view.content).onHide) === null || _b === void 0 ? void 0 : _b.call(_a);
             }
         }
         this.panel = undefined;
@@ -6052,7 +6050,7 @@ class LongPressDetector extends CompositeDisposable {
         const delay = (_b = this.options.delay) !== null && _b !== void 0 ? _b : DEFAULT_DELAY;
         const tolerance = (_c = this.options.tolerance) !== null && _c !== void 0 ? _c : DEFAULT_TOLERANCE;
         // Source's owning window — popout drags fire on their own window.
-        const targetWindow = (_e = (_d = this.element.ownerDocument) === null || _d === void 0 ? void 0 : _d.defaultView) !== null && _e !== void 0 ? _e : window;
+        const targetWindow = (_e = (_d = this.element.ownerDocument) === null || _d === void 0 ? void 0 : _d.defaultView) !== null && _e !== void 0 ? _e : globalThis.window;
         this._timer = setTimeout(() => {
             this._timer = undefined;
             this._cancelPending();
@@ -6171,14 +6169,14 @@ function resolveDndCapabilities(options) {
     }
 }
 function isCoarsePrimaryInput$1() {
-    if (typeof window === 'undefined' || !window.matchMedia) {
+    if (globalThis.window === undefined || !globalThis.matchMedia) {
         return false;
     }
     // Coarse pointer without any fine pointer = phone-class device. A laptop
     // touchscreen reports both, and we want HTML5 to remain available there
     // because a real mouse is also plugged in.
-    const coarse = window.matchMedia('(pointer: coarse)').matches;
-    const fine = window.matchMedia('(pointer: fine)').matches;
+    const coarse = globalThis.matchMedia('(pointer: coarse)').matches;
+    const fine = globalThis.matchMedia('(pointer: fine)').matches;
     return coarse && !fine;
 }
 
@@ -6232,7 +6230,7 @@ class Tab extends CompositeDisposable {
                 return false;
             }
             const data = getPanelData();
-            if (data && this.accessor.id === data.viewId) {
+            if (this.accessor.id === (data === null || data === void 0 ? void 0 : data.viewId)) {
                 // Smooth-reorder takes over the in-flight visual when active,
                 // so individual tab overlays are suppressed for internal drags.
                 if (((_a = this.accessor.options.theme) === null || _a === void 0 ? void 0 : _a.tabAnimation) === 'smooth') {
@@ -6333,7 +6331,7 @@ class Tab extends CompositeDisposable {
     }
     setContent(part) {
         if (this.content) {
-            this._element.removeChild(this.content.element);
+            this.content.element.remove();
         }
         this.content = part;
         this._element.appendChild(this.content.element);
@@ -6491,7 +6489,7 @@ class GroupDragSource extends CompositeDisposable {
         this.addDisposables(this._onDragStart);
         const buildMultiPanelsGhost = () => {
             const ghostEl = document.createElement('div');
-            const style = window.getComputedStyle(this._element);
+            const style = globalThis.getComputedStyle(this._element);
             const bgColor = style.getPropertyValue('--dv-activegroup-visiblepanel-tab-background-color');
             const color = style.getPropertyValue('--dv-activegroup-visiblepanel-tab-color');
             ghostEl.style.backgroundColor = bgColor;
@@ -6673,7 +6671,7 @@ class VoidContainer extends CompositeDisposable {
                 return false;
             }
             const data = getPanelData();
-            if (data && this.accessor.id === data.viewId) {
+            if (this.accessor.id === (data === null || data === void 0 ? void 0 : data.viewId)) {
                 return true;
             }
             return group.model.canDisplayOverlay(event, position, 'header_space');
@@ -6768,15 +6766,16 @@ class Scrollbar extends CompositeDisposable {
                 this._scrollOffset = originalScrollOffset + delta / p;
                 this.calculateScrollbarStyles();
             };
+            const doc = this.element.ownerDocument;
             const onEnd = () => {
                 toggleClass(this.element, 'dv-scrollable-scrolling', false);
-                document.removeEventListener('pointermove', onPointerMove);
-                document.removeEventListener('pointerup', onEnd);
-                document.removeEventListener('pointercancel', onEnd);
+                doc.removeEventListener('pointermove', onPointerMove);
+                doc.removeEventListener('pointerup', onEnd);
+                doc.removeEventListener('pointercancel', onEnd);
             };
-            document.addEventListener('pointermove', onPointerMove);
-            document.addEventListener('pointerup', onEnd);
-            document.addEventListener('pointercancel', onEnd);
+            doc.addEventListener('pointermove', onPointerMove);
+            doc.addEventListener('pointerup', onEnd);
+            doc.addEventListener('pointercancel', onEnd);
         }), addDisposableListener(this.element, 'scroll', () => {
             this.calculateScrollbarStyles();
         }), addDisposableListener(this.scrollableElement, 'scroll', () => {
@@ -6936,9 +6935,7 @@ let _fallbackPalette;
  * palette through.
  */
 function getFallbackPalette() {
-    if (!_fallbackPalette) {
-        _fallbackPalette = new TabGroupColorPalette(DEFAULT_TAB_GROUP_COLORS, true);
-    }
+    _fallbackPalette !== null && _fallbackPalette !== void 0 ? _fallbackPalette : (_fallbackPalette = new TabGroupColorPalette(DEFAULT_TAB_GROUP_COLORS, true));
     return _fallbackPalette;
 }
 /**
@@ -7180,8 +7177,7 @@ class BaseTabGroupIndicator {
             const isAnimating = tg.collapsed ||
                 tg.panelIds.some((pid) => {
                     const te = tabMap.get(pid);
-                    return (te &&
-                        te.value.element.classList.contains('dv-tab--group-expanding'));
+                    return te === null || te === void 0 ? void 0 : te.value.element.classList.contains('dv-tab--group-expanding');
                 });
             if (isAnimating && chipEl) {
                 const chipRect = chipEl.getBoundingClientRect();
@@ -7278,7 +7274,10 @@ class WrapTabGroupIndicator extends BaseTabGroupIndicator {
         // Ensure SVG + path child exists (created once, reused)
         let svg = underline.firstElementChild;
         let path;
-        if (!svg || svg.tagName !== 'svg') {
+        if ((svg === null || svg === void 0 ? void 0 : svg.tagName) === 'svg') {
+            path = svg.firstElementChild;
+        }
+        else {
             underline.replaceChildren();
             svg = document.createElementNS('http://www.w3.org/2000/svg', 'svg');
             svg.style.display = 'block';
@@ -7287,9 +7286,6 @@ class WrapTabGroupIndicator extends BaseTabGroupIndicator {
             svg.appendChild(path);
             underline.appendChild(svg);
         }
-        else {
-            path = svg.firstElementChild;
-        }
         path.setAttribute('stroke', color);
         path.setAttribute('stroke-width', String(t));
         if (!activeTabEntry) {
@@ -7426,6 +7422,7 @@ class TabGroupManager {
      * in the parent group model. Call after any tab group mutation.
      */
     update() {
+        var _a;
         const model = this._ctx.group.model;
         const tabGroups = model.getTabGroups();
         // Track which group IDs are still active
@@ -7440,6 +7437,7 @@ class TabGroupManager {
             if (!activeGroupIds.has(groupId)) {
                 entry.chip.element.remove();
                 entry.chip.dispose();
+                (_a = entry.dragSourcesDisposable) === null || _a === void 0 ? void 0 : _a.dispose();
                 entry.disposable.dispose();
                 this._chipRenderers.delete(groupId);
             }
@@ -7568,12 +7566,13 @@ class TabGroupManager {
         this._pendingTransitionCleanups.delete(panelId);
     }
     updateDragAndDropState() {
+        var _a, _b, _c;
         const caps = resolveDndCapabilities(this._ctx.accessor.options);
         for (const entry of this._chipRenderers.values()) {
             entry.chip.element.draggable = caps.html5;
-            entry.html5DragSource.setDisabled(!caps.html5);
-            entry.pointerDragSource.setDisabled(!caps.pointer);
-            entry.pointerDragSource.setTouchOnly(!caps.pointerHandlesMouse);
+            (_a = entry.html5DragSource) === null || _a === void 0 ? void 0 : _a.setDisabled(!caps.html5);
+            (_b = entry.pointerDragSource) === null || _b === void 0 ? void 0 : _b.setDisabled(!caps.pointer);
+            (_c = entry.pointerDragSource) === null || _c === void 0 ? void 0 : _c.setTouchOnly(!caps.pointerHandlesMouse);
         }
     }
     /**
@@ -7582,19 +7581,28 @@ class TabGroupManager {
      * iframe shield are released BEFORE the cross-group move detaches
      * the chip (chip dispose is scheduled on a microtask via
      * `_scheduleTabGroupUpdate`, which is too late for callers that read
-     * `getPanelData()` synchronously after the move). Idempotent — the
-     * subsequent `update()` will also dispose the sources.
+     * `getPanelData()` synchronously after the move).
+     *
+     * A cross-group move dissolves the source chip entirely, but a
+     * within-group reorder keeps the same chip element — so the sources are
+     * cleared to `undefined` here and the subsequent `update()` re-arms them
+     * via `_ensureChipForGroup`. Without that re-arm the chip would fire a
+     * native `dragstart` (the element keeps `draggable=true`) but never set
+     * a transfer payload, leaving the group permanently undraggable after
+     * its first move. (#1410)
      */
     disposeChipDrag(tabGroupId) {
-        var _a, _b;
+        var _a;
         const entry = this._chipRenderers.get(tabGroupId);
         if (!entry) {
             return;
         }
         // Optional-chained because tests may inject minimal entries
         // that skip the manager's normal `_ensureChipForGroup` flow.
-        (_a = entry.html5DragSource) === null || _a === void 0 ? void 0 : _a.dispose();
-        (_b = entry.pointerDragSource) === null || _b === void 0 ? void 0 : _b.dispose();
+        (_a = entry.dragSourcesDisposable) === null || _a === void 0 ? void 0 : _a.dispose();
+        entry.html5DragSource = undefined;
+        entry.pointerDragSource = undefined;
+        entry.dragSourcesDisposable = undefined;
     }
     /** Cloned chip rect used as the pointer follow-finger ghost. */
     _buildChipGhostElement(chipEl) {
@@ -7607,7 +7615,7 @@ class TabGroupManager {
         return clone;
     }
     disposeAll() {
-        var _a;
+        var _a, _b;
         (_a = this._indicator) === null || _a === void 0 ? void 0 : _a.dispose();
         this._indicator = null;
         for (const [, cleanup] of this._pendingTransitionCleanups) {
@@ -7617,12 +7625,13 @@ class TabGroupManager {
         for (const [, entry] of this._chipRenderers) {
             entry.chip.element.remove();
             entry.chip.dispose();
+            (_b = entry.dragSourcesDisposable) === null || _b === void 0 ? void 0 : _b.dispose();
             entry.disposable.dispose();
         }
         this._chipRenderers.clear();
     }
     _ensureIndicator() {
-        var _a, _b;
+        var _a, _b, _c;
         const mode = (_b = (_a = this._ctx.accessor.options.theme) === null || _a === void 0 ? void 0 : _a.tabGroupIndicator) !== null && _b !== void 0 ? _b : 'wrap';
         const Ctor = mode === 'none' ? NoneTabGroupIndicator : WrapTabGroupIndicator;
         // Re-create if the indicator type changed (e.g. theme switch)
@@ -7630,28 +7639,24 @@ class TabGroupManager {
             this._indicator.dispose();
             this._indicator = null;
         }
-        if (!this._indicator) {
-            this._indicator = new Ctor({
-                tabsList: this._ctx.tabsList,
-                getTabGroups: () => this._ctx.group.model.getTabGroups(),
-                getActivePanelId: () => { var _a; return (_a = this._ctx.group.activePanel) === null || _a === void 0 ? void 0 : _a.id; },
-                getTabMap: () => this._ctx.getTabMap(),
-                getChipElement: (id) => { var _a; return (_a = this._chipRenderers.get(id)) === null || _a === void 0 ? void 0 : _a.chip.element; },
-                getDirection: () => this._ctx.getDirection(),
-                getHeaderPosition: () => this._ctx.group.model.headerPosition,
-                getColorPalette: () => this._ctx.accessor.tabGroupColorPalette,
-            });
-        }
+        (_c = this._indicator) !== null && _c !== void 0 ? _c : (this._indicator = new Ctor({
+            tabsList: this._ctx.tabsList,
+            getTabGroups: () => this._ctx.group.model.getTabGroups(),
+            getActivePanelId: () => { var _a; return (_a = this._ctx.group.activePanel) === null || _a === void 0 ? void 0 : _a.id; },
+            getTabMap: () => this._ctx.getTabMap(),
+            getChipElement: (id) => { var _a; return (_a = this._chipRenderers.get(id)) === null || _a === void 0 ? void 0 : _a.chip.element; },
+            getDirection: () => this._ctx.getDirection(),
+            getHeaderPosition: () => this._ctx.group.model.headerPosition,
+            getColorPalette: () => this._ctx.accessor.tabGroupColorPalette,
+        }));
     }
-    _ensureChipForGroup(tabGroup) {
-        if (this._chipRenderers.has(tabGroup.id)) {
-            return;
-        }
-        const createChip = this._ctx.accessor.options.createTabGroupChipComponent;
-        const chip = createChip
-            ? createChip(tabGroup)
-            : new TabGroupChip(this._ctx.accessor.tabGroupColorPalette);
-        chip.init({ tabGroup, api: this._ctx.accessor.api });
+    /**
+     * Create the HTML5 + pointer drag sources for a chip (and the direct
+     * dragend listener that clears the transfer synchronously). Extracted so
+     * both the initial chip build and the post-move re-arm in
+     * `_ensureChipForGroup` share one wiring path. (#1410)
+     */
+    _createChipDragSources(chip, tabGroup) {
         const caps = resolveDndCapabilities(this._ctx.accessor.options);
         chip.element.draggable = caps.html5;
         const panelTransfer = LocalSelectionTransfer.getInstance();
@@ -7703,9 +7708,12 @@ class TabGroupManager {
         // the listener survives chip disposal in the detach-then-dragend
         // cross-group path; `once: true` auto-removes after the single
         // dragend that we care about. (#1254)
-        chip.element.addEventListener('dragend', () => {
+        const onDragEndClear = () => {
             panelTransfer.clearData(PanelTransfer.prototype);
-        }, { once: true });
+        };
+        chip.element.addEventListener('dragend', onDragEndClear, {
+            once: true,
+        });
         const pointerDragSource = pointerBackend.createDragSource(chip.element, {
             getData,
             disabled: !caps.pointer,
@@ -7720,6 +7728,41 @@ class TabGroupManager {
                 this._callbacks.onChipDragStart(tabGroup, chip, event);
             },
         });
+        return {
+            html5DragSource,
+            pointerDragSource,
+            disposable: {
+                dispose: () => {
+                    html5DragSource.dispose();
+                    pointerDragSource.dispose();
+                    chip.element.removeEventListener('dragend', onDragEndClear);
+                },
+            },
+        };
+    }
+    _ensureChipForGroup(tabGroup) {
+        const existing = this._chipRenderers.get(tabGroup.id);
+        if (existing) {
+            // A committed group move tears down the chip's drag sources via
+            // `disposeChipDrag`, but a within-group reorder keeps the same
+            // chip element alive. Re-arm the sources so the group stays
+            // draggable; without this the chip is stuck after its first
+            // move — `dragstart` still fires (the element keeps
+            // `draggable=true`) but no transfer payload is set. (#1410)
+            if (!existing.html5DragSource) {
+                const sources = this._createChipDragSources(existing.chip, tabGroup);
+                existing.html5DragSource = sources.html5DragSource;
+                existing.pointerDragSource = sources.pointerDragSource;
+                existing.dragSourcesDisposable = sources.disposable;
+            }
+            return;
+        }
+        const createChip = this._ctx.accessor.options.createTabGroupChipComponent;
+        const chip = createChip
+            ? createChip(tabGroup)
+            : new TabGroupChip(this._ctx.accessor.tabGroupColorPalette);
+        chip.init({ tabGroup, api: this._ctx.accessor.api });
+        const dragSources = this._createChipDragSources(chip, tabGroup);
         const disposables = [
             tabGroup.onDidChange(() => {
                 var _a;
@@ -7733,17 +7776,19 @@ class TabGroupManager {
             tabGroup.onDidCollapseChange(() => {
                 this._updateTabGroupClasses();
             }),
-            html5DragSource,
-            pointerDragSource,
         ];
         // Context menu: built-in TabGroupChip already aggregates right-click
         // + touch long-press into `onContextMenu`. Custom chip renderers
         // don't, so attach a long-press detector and contextmenu listener
         // directly on their element.
         const onContextMenu = (event) => {
+            var _a, _b;
             // A long-press on a chip should preempt the in-flight pointer
-            // drag and open the menu instead.
-            pointerDragSource.cancelPending();
+            // drag and open the menu instead. Resolve the source from the
+            // entry so a post-move re-armed source is cancelled, not the
+            // stale one captured at build time. (#1410)
+            (_b = (_a = this._chipRenderers
+                .get(tabGroup.id)) === null || _a === void 0 ? void 0 : _a.pointerDragSource) === null || _b === void 0 ? void 0 : _b.cancelPending();
             this._callbacks.onChipContextMenu(tabGroup, event);
         };
         if (chip instanceof TabGroupChip) {
@@ -7776,7 +7821,7 @@ class TabGroupManager {
                     return false;
                 }
                 const data = getPanelData();
-                if (data && this._ctx.accessor.id === data.viewId) {
+                if (this._ctx.accessor.id === (data === null || data === void 0 ? void 0 : data.viewId)) {
                     if (((_a = this._ctx.accessor.options.theme) === null || _a === void 0 ? void 0 : _a.tabAnimation) ===
                         'smooth') {
                         return false;
@@ -7792,8 +7837,9 @@ class TabGroupManager {
         const disposable = new CompositeDisposable(...disposables);
         this._chipRenderers.set(tabGroup.id, {
             chip,
-            html5DragSource,
-            pointerDragSource,
+            html5DragSource: dragSources.html5DragSource,
+            pointerDragSource: dragSources.pointerDragSource,
+            dragSourcesDisposable: dragSources.disposable,
             disposable,
             dropTarget,
         });
@@ -8397,12 +8443,10 @@ class Tabs extends CompositeDisposable {
                         parentElement.scrollTop = running;
                     }
                 }
-                else {
-                    if (running < parentElement.scrollLeft ||
-                        running + element.clientWidth >
-                            parentElement.scrollLeft + parentElement.clientWidth) {
-                        parentElement.scrollLeft = running;
-                    }
+                else if (running < parentElement.scrollLeft ||
+                    running + element.clientWidth >
+                        parentElement.scrollLeft + parentElement.clientWidth) {
+                    parentElement.scrollLeft = running;
                 }
             }
             running += isVertical
@@ -8514,15 +8558,13 @@ class Tabs extends CompositeDisposable {
                 });
                 return;
             }
-            switch (event.button) {
-                case 0:
-                    if (this.group.api.location.type === 'edge') ;
-                    else {
-                        if (this.group.activePanel !== panel) {
-                            this.group.model.openPanel(panel);
-                        }
-                    }
-                    break;
+            // Edge groups are skipped here: all their tab interaction
+            // is handled by onTabClick to avoid race conditions with
+            // active panel state.
+            if (event.button === 0 &&
+                this.group.api.location.type !== 'edge' &&
+                this.group.activePanel !== panel) {
+                this.group.model.openPanel(panel);
             }
         }), tab.onDrop((event) => {
             var _a, _b, _c, _d;
@@ -8545,12 +8587,12 @@ class Tabs extends CompositeDisposable {
                     targetTabGroupId: animState.targetTabGroupId,
                 });
                 if (((_b = this.accessor.options.theme) === null || _b === void 0 ? void 0 : _b.tabAnimation) === 'smooth') {
-                    this.runFlipAnimation(firstPositions, animState.sourceTabId, animState.sourceIndex === -1, animState.sourceIndex !== -1
-                        ? {
+                    this.runFlipAnimation(firstPositions, animState.sourceTabId, animState.sourceIndex === -1, animState.sourceIndex === -1
+                        ? undefined
+                        : {
                             from: Math.min(animState.sourceIndex, dropIndex),
                             to: Math.max(animState.sourceIndex, dropIndex),
-                        }
-                        : undefined);
+                        });
                 }
             }
             else {
@@ -8800,7 +8842,7 @@ class Tabs extends CompositeDisposable {
             return;
         }
         const data = getPanelData();
-        const sourceIndex = data && data.groupId === this.group.id && data.panelId
+        const sourceIndex = (data === null || data === void 0 ? void 0 : data.groupId) === this.group.id && (data === null || data === void 0 ? void 0 : data.panelId)
             ? this._tabs.findIndex((x) => x.value.panel.id === data.panelId)
             : -1;
         const adjustedIndex = insertionIndex -
@@ -10158,9 +10200,9 @@ class TabGroup extends CompositeDisposable {
         if (this._panelIds.includes(panelId)) {
             return;
         }
-        const insertIndex = index !== undefined
-            ? Math.max(0, Math.min(index, this._panelIds.length))
-            : this._panelIds.length;
+        const insertIndex = index === undefined
+            ? this._panelIds.length
+            : Math.max(0, Math.min(index, this._panelIds.length));
         this._panelIds.splice(insertIndex, 0, panelId);
         this._onDidPanelChange.fire({ panelId, type: 'add' });
     }
@@ -10608,7 +10650,7 @@ class DockviewGroupPanelModel extends CompositeDisposable {
         }
         // Remove from any existing group first
         const existingGroup = this.getTabGroupForPanel(panelId);
-        if (existingGroup && existingGroup.id === tabGroupId) {
+        if ((existingGroup === null || existingGroup === void 0 ? void 0 : existingGroup.id) === tabGroupId) {
             return; // already in this group — no mutation
         }
         this._bracketTabGroupMutation(() => {
@@ -10629,7 +10671,7 @@ class DockviewGroupPanelModel extends CompositeDisposable {
      */
     movePanelWithinGroup(tabGroupId, panelId, newIndex) {
         const tabGroup = this._tabGroupMap.get(tabGroupId);
-        if (!tabGroup || !tabGroup.containsPanel(panelId)) {
+        if (!(tabGroup === null || tabGroup === void 0 ? void 0 : tabGroup.containsPanel(panelId))) {
             return;
         }
         // Remove and re-add at new index within the group
@@ -10842,7 +10884,7 @@ class DockviewGroupPanelModel extends CompositeDisposable {
         for (let i = activePanelIndex + 1; i < this._panels.length; i++) {
             const candidate = this._panels[i];
             const candidateGroup = this._findTabGroupForPanel(candidate.id);
-            if (!candidateGroup || !candidateGroup.collapsed) {
+            if (!(candidateGroup === null || candidateGroup === void 0 ? void 0 : candidateGroup.collapsed)) {
                 this.doSetActivePanel(candidate);
                 this.updateContainer();
                 return;
@@ -10851,7 +10893,7 @@ class DockviewGroupPanelModel extends CompositeDisposable {
         for (let i = activePanelIndex - 1; i >= 0; i--) {
             const candidate = this._panels[i];
             const candidateGroup = this._findTabGroupForPanel(candidate.id);
-            if (!candidateGroup || !candidateGroup.collapsed) {
+            if (!(candidateGroup === null || candidateGroup === void 0 ? void 0 : candidateGroup.collapsed)) {
                 this.doSetActivePanel(candidate);
                 this.updateContainer();
                 return;
@@ -10881,9 +10923,9 @@ class DockviewGroupPanelModel extends CompositeDisposable {
     restoreTabGroups(serializedGroups) {
         // Bump counter past any restored numeric suffixes to avoid ID collisions
         for (const data of serializedGroups) {
-            const match = data.id.match(/-(\d+)$/);
+            const match = /-(\d+)$/.exec(data.id);
             if (match) {
-                const num = parseInt(match[1], 10) + 1;
+                const num = Number.parseInt(match[1], 10) + 1;
                 if (num > this._tabGroupIdCounter) {
                     this._tabGroupIdCounter = num;
                 }
@@ -11000,32 +11042,26 @@ class DockviewGroupPanelModel extends CompositeDisposable {
         return result;
     }
     moveToNext(options) {
-        if (!options) {
-            options = {};
-        }
-        if (!options.panel) {
-            options.panel = this.activePanel;
-        }
+        var _a;
+        options !== null && options !== void 0 ? options : (options = {});
+        (_a = options.panel) !== null && _a !== void 0 ? _a : (options.panel = this.activePanel);
         const index = options.panel ? this.panels.indexOf(options.panel) : -1;
         let normalizedIndex;
         if (index < this.panels.length - 1) {
             normalizedIndex = index + 1;
         }
-        else if (!options.suppressRoll) {
-            normalizedIndex = 0;
+        else if (options.suppressRoll) {
+            return;
         }
         else {
-            return;
+            normalizedIndex = 0;
         }
         this.openPanel(this.panels[normalizedIndex]);
     }
     moveToPrevious(options) {
-        if (!options) {
-            options = {};
-        }
-        if (!options.panel) {
-            options.panel = this.activePanel;
-        }
+        var _a;
+        options !== null && options !== void 0 ? options : (options = {});
+        (_a = options.panel) !== null && _a !== void 0 ? _a : (options.panel = this.activePanel);
         if (!options.panel) {
             return;
         }
@@ -11034,11 +11070,11 @@ class DockviewGroupPanelModel extends CompositeDisposable {
         if (index > 0) {
             normalizedIndex = index - 1;
         }
-        else if (!options.suppressRoll) {
-            normalizedIndex = this.panels.length - 1;
+        else if (options.suppressRoll) {
+            return;
         }
         else {
-            return;
+            normalizedIndex = this.panels.length - 1;
         }
         this.openPanel(this.panels[normalizedIndex]);
     }
@@ -11138,7 +11174,7 @@ class DockviewGroupPanelModel extends CompositeDisposable {
         if (!this._activePanel && this.panels.length > 0) {
             const candidate = this._panels.find((p) => {
                 const tg = this._findTabGroupForPanel(p.id);
-                return !tg || !tg.collapsed;
+                return !(tg === null || tg === void 0 ? void 0 : tg.collapsed);
             });
             if (candidate) {
                 this.doSetActivePanel(candidate);
@@ -11677,7 +11713,7 @@ class DockviewGroupPanelApiImpl extends GridviewPanelApiImpl {
     getWindow() {
         return this.location.type === 'popout'
             ? this.location.getWindow()
-            : window;
+            : globalThis.window;
     }
     setHeaderPosition(position) {
         if (!this._group) {
@@ -12260,9 +12296,8 @@ class DockviewPanel extends CompositeDisposable {
         }
     }
     update(event) {
-        var _a;
         // merge the new parameters with the existing parameters
-        this._params = Object.assign(Object.assign({}, ((_a = this._params) !== null && _a !== void 0 ? _a : {})), event.params);
+        this._params = Object.assign(Object.assign({}, this._params), event.params);
         /**
          * delete new keys that have a value of undefined,
          * allow values of null
@@ -12659,19 +12694,19 @@ class Overlay extends CompositeDisposable {
         const element = this._element.getBoundingClientRect();
         const result = {};
         if (this.verticalAlignment === 'top') {
-            result.top = parseFloat(this._element.style.top);
+            result.top = Number.parseFloat(this._element.style.top);
         }
         else if (this.verticalAlignment === 'bottom') {
-            result.bottom = parseFloat(this._element.style.bottom);
+            result.bottom = Number.parseFloat(this._element.style.bottom);
         }
         else {
             result.top = element.top - container.top;
         }
         if (this.horiziontalAlignment === 'left') {
-            result.left = parseFloat(this._element.style.left);
+            result.left = Number.parseFloat(this._element.style.left);
         }
         else if (this.horiziontalAlignment === 'right') {
-            result.right = parseFloat(this._element.style.right);
+            result.right = Number.parseFloat(this._element.style.right);
         }
         else {
             result.left = element.left - container.left;
@@ -12735,7 +12770,7 @@ class Overlay extends CompositeDisposable {
                         }
                     }
                 },
-            }, addDisposableListener(window, 'pointermove', (e) => {
+            }, addDisposableListener(globalThis.window, 'pointermove', (e) => {
                 if (this._dragCancelled) {
                     return;
                 }
@@ -12744,12 +12779,10 @@ class Overlay extends CompositeDisposable {
                 const y = e.clientY - containerRect.top;
                 toggleClass(this._element, 'dv-resize-container-dragging', true);
                 const overlayRect = this._element.getBoundingClientRect();
-                if (offset === null) {
-                    offset = {
-                        x: e.clientX - overlayRect.left,
-                        y: e.clientY - overlayRect.top,
-                    };
-                }
+                offset !== null && offset !== void 0 ? offset : (offset = {
+                    x: e.clientX - overlayRect.left,
+                    y: e.clientY - overlayRect.top,
+                });
                 const xOffset = Math.max(0, this.getMinimumWidth(overlayRect.width));
                 const yOffset = Math.max(0, this.getMinimumHeight(overlayRect.height));
                 let proposedTop = y - offset.y;
@@ -12801,7 +12834,7 @@ class Overlay extends CompositeDisposable {
                     hasMoved = true;
                     this._onDidStartMoving.fire();
                 }
-            }), addDisposableListener(window, 'pointerup', end), addDisposableListener(window, 'pointercancel', end));
+            }), addDisposableListener(globalThis.window, 'pointerup', end), addDisposableListener(globalThis.window, 'pointercancel', end));
         };
         this.addDisposables(addDisposableListener(dragTarget, 'pointerdown', (event) => {
             if (event.defaultPrevented) {
@@ -12855,20 +12888,18 @@ class Overlay extends CompositeDisposable {
                 move.dispose();
                 this._onDidChangeEnd.fire();
             };
-            move.value = new CompositeDisposable(addDisposableListener(window, 'pointermove', (e) => {
+            move.value = new CompositeDisposable(addDisposableListener(globalThis.window, 'pointermove', (e) => {
                 const containerRect = this.options.container.getBoundingClientRect();
                 const overlayRect = this._element.getBoundingClientRect();
                 const y = e.clientY - containerRect.top;
                 const x = e.clientX - containerRect.left;
-                if (startPosition === null) {
-                    // record the initial dimensions since as all subsequence moves are relative to this
-                    startPosition = {
-                        originalY: y,
-                        originalHeight: overlayRect.height,
-                        originalX: x,
-                        originalWidth: overlayRect.width,
-                    };
-                }
+                // record the initial dimensions since as all subsequence moves are relative to this
+                startPosition !== null && startPosition !== void 0 ? startPosition : (startPosition = {
+                    originalY: y,
+                    originalHeight: overlayRect.height,
+                    originalX: x,
+                    originalWidth: overlayRect.width,
+                });
                 let top = undefined;
                 let bottom = undefined;
                 let height = undefined;
@@ -12883,7 +12914,8 @@ class Overlay extends CompositeDisposable {
                         ? Math.max(0, containerRect.height -
                             Overlay.MINIMUM_HEIGHT)
                         : Math.max(0, startPosition.originalY +
-                            startPosition.originalHeight -
+                            startPosition
+                                .originalHeight -
                             Overlay.MINIMUM_HEIGHT);
                     top = clamp(y, 0, maxTop);
                     height =
@@ -12898,8 +12930,8 @@ class Overlay extends CompositeDisposable {
                             startPosition.originalHeight;
                     // When dragging bottom handle, constrain height to container height
                     const minHeight = top < 0 &&
-                        typeof this.options.minimumInViewportHeight ===
-                            'number'
+                        typeof this.options
+                            .minimumInViewportHeight === 'number'
                         ? -top +
                             this.options.minimumInViewportHeight
                         : Overlay.MINIMUM_HEIGHT;
@@ -12929,8 +12961,8 @@ class Overlay extends CompositeDisposable {
                             startPosition.originalWidth;
                     // When dragging right handle, constrain width to container width
                     const minWidth = left < 0 &&
-                        typeof this.options.minimumInViewportWidth ===
-                            'number'
+                        typeof this.options
+                            .minimumInViewportWidth === 'number'
                         ? -left +
                             this.options.minimumInViewportWidth
                         : Overlay.MINIMUM_WIDTH;
@@ -12999,7 +13031,7 @@ class Overlay extends CompositeDisposable {
                         }
                     }
                 },
-            }, addDisposableListener(window, 'pointerup', end), addDisposableListener(window, 'pointercancel', end));
+            }, addDisposableListener(globalThis.window, 'pointerup', end), addDisposableListener(globalThis.window, 'pointercancel', end));
         }));
     }
     getMinimumWidth(width) {
@@ -14113,7 +14145,7 @@ class PositionCache {
     }
     getPosition(element) {
         const cached = this.cache.get(element);
-        if (cached && cached.frameId === this.currentFrameId) {
+        if ((cached === null || cached === void 0 ? void 0 : cached.frameId) === this.currentFrameId) {
             return cached.rect;
         }
         this.scheduleFrameUpdate();
@@ -14325,11 +14357,10 @@ class OverlayRenderContainer extends CompositeDisposable {
             correctLayerPosition();
         }));
         this.map[panel.api.id].destroy = Disposable.from(() => {
-            var _a;
             if (contentElement.parentElement === focusContainer) {
-                focusContainer.removeChild(contentElement);
+                contentElement.remove();
             }
-            (_a = focusContainer.parentElement) === null || _a === void 0 ? void 0 : _a.removeChild(focusContainer);
+            focusContainer.remove();
         });
         correctLayerPosition();
         queueMicrotask(() => {
@@ -14394,13 +14425,13 @@ typeof SuppressedError === "function" ? SuppressedError : function (error, suppr
 function assertSameOriginPopoutUrl(url) {
     let resolved;
     try {
-        resolved = new URL(url, window.location.href);
+        resolved = new URL(url, globalThis.location.href);
     }
     catch (_a) {
         throw new Error(`dockview: invalid popout URL: ${url}`);
     }
     const protocolOk = resolved.protocol === 'http:' || resolved.protocol === 'https:';
-    if (!protocolOk || resolved.origin !== window.location.origin) {
+    if (!protocolOk || resolved.origin !== globalThis.location.origin) {
         throw new Error(`dockview: popout URL must be same-origin http(s); got: ${url}`);
     }
 }
@@ -14478,7 +14509,7 @@ class PopoutWindow extends CompositeDisposable {
             this._window = { value: externalWindow, disposable };
             disposable.addDisposables(Disposable.from(() => {
                 externalWindow.close();
-            }), addDisposableListener(window, 'beforeunload', () => {
+            }), addDisposableListener(globalThis.window, 'beforeunload', () => {
                 /**
                  * before the main window closes we should close this popup too
                  * to be good citizens
@@ -14508,7 +14539,7 @@ class PopoutWindow extends CompositeDisposable {
                         const externalDocument = externalWindow.document;
                         externalDocument.title = document.title;
                         externalDocument.body.appendChild(container);
-                        addStyles(externalDocument, window.document.styleSheets, {
+                        addStyles(externalDocument, globalThis.document.styleSheets, {
                             nonce: this.options.nonce,
                         });
                         /**
@@ -14561,11 +14592,11 @@ class StrictEventsSequencing extends CompositeDisposable {
                 panels.add(panel.api.id);
             }
         }), this.accessor.onDidRemovePanel((panel) => {
-            if (!panels.has(panel.api.id)) {
-                throw new Error(`dockview: Invalid event sequence. [onDidRemovePanel] called for panel ${panel.api.id} but panel does not exists`);
+            if (panels.has(panel.api.id)) {
+                panels.delete(panel.api.id);
             }
             else {
-                panels.delete(panel.api.id);
+                throw new Error(`dockview: Invalid event sequence. [onDidRemovePanel] called for panel ${panel.api.id} but panel does not exists`);
             }
         }), this.accessor.onDidAddGroup((group) => {
             if (groups.has(group.api.id)) {
@@ -14575,11 +14606,11 @@ class StrictEventsSequencing extends CompositeDisposable {
                 groups.add(group.api.id);
             }
         }), this.accessor.onDidRemoveGroup((group) => {
-            if (!groups.has(group.api.id)) {
-                throw new Error(`dockview: Invalid event sequence. [onDidRemoveGroup] called for group ${group.api.id} but group does not exists`);
+            if (groups.has(group.api.id)) {
+                groups.delete(group.api.id);
             }
             else {
-                groups.delete(group.api.id);
+                throw new Error(`dockview: Invalid event sequence. [onDidRemoveGroup] called for group ${group.api.id} but group does not exists`);
             }
         }));
     }
@@ -14594,7 +14625,7 @@ function isCoarsePrimaryInput(win) {
     return coarse && !fine;
 }
 class PopupService extends CompositeDisposable {
-    constructor(root, win = window) {
+    constructor(root, win = globalThis.window) {
         super();
         this._active = null;
         this._activeDisposable = new MutableDisposable();
@@ -14711,9 +14742,8 @@ class DropTargetAnchorContainer extends CompositeDisposable {
         }
         return {
             clear: () => {
-                var _a;
                 if (this._model) {
-                    (_a = this._model.root.parentElement) === null || _a === void 0 ? void 0 : _a.removeChild(this._model.root);
+                    this._model.root.remove();
                 }
                 this._model = undefined;
             },
@@ -14766,16 +14796,30 @@ class DropTargetAnchorContainer extends CompositeDisposable {
 }
 
 class EdgeGroupView {
+    /**
+     * The effective (pre-gap) collapsed size: the measured tab-strip size when
+     * available, otherwise the configured/theme base.
+     */
+    get _effectiveBaseCollapsed() {
+        var _a;
+        return (_a = this._measuredTabSize) !== null && _a !== void 0 ? _a : this._baseCollapsedSize;
+    }
     get minimumSize() {
         // When collapsed, lock size to collapsedSize so sash can't drag it open
-        return this._isCollapsed
-            ? this._collapsedSize
-            : this._expandedMinimumSize;
+        if (this._isCollapsed) {
+            return this.collapsedSize;
+        }
+        // If the caller explicitly provides a minimumSize, respect it.
+        // Otherwise fall back to collapsedSize + 50 so the expanded state is
+        // visually distinguishable from the collapsed state.
+        return this._baseMinimumSize !== undefined
+            ? this._baseMinimumSize + this._gapAdd
+            : this._effectiveBaseCollapsed + 50 + this._gapAdd;
     }
     get maximumSize() {
         // When collapsed, lock size to collapsedSize so sash can't drag it open
         return this._isCollapsed
-            ? this._collapsedSize
+            ? this.collapsedSize
             : this._expandedMaximumSize;
     }
     get element() {
@@ -14788,34 +14832,62 @@ class EdgeGroupView {
         return this._lastExpandedSize;
     }
     get collapsedSize() {
-        return this._collapsedSize;
+        return this._effectiveBaseCollapsed + this._gapAdd;
     }
-    constructor(options, group, orientation) {
+    constructor(options, group, orientation, gapAdd = 0) {
         var _a, _b, _c;
         this._onDidChange = new Emitter();
         this.onDidChange = this._onDidChange.event;
         this.snap = false;
         this.priority = LayoutPriority.Low;
         this._isCollapsed = false;
+        this._tabSizeDisposables = new CompositeDisposable();
         this._group = group;
         this._orientation = orientation;
         group.element.classList.add('dv-edge-group');
         group.element.dataset.testid = `dv-edge-group-${options.id}`;
-        this._collapsedSize = (_a = options.collapsedSize) !== null && _a !== void 0 ? _a : 35;
+        this._baseCollapsedSize = (_a = options.collapsedSize) !== null && _a !== void 0 ? _a : 35;
+        this._baseMinimumSize = options.minimumSize;
+        this._gapAdd = gapAdd;
         this._expandedMaximumSize =
             (_b = options.maximumSize) !== null && _b !== void 0 ? _b : Number.POSITIVE_INFINITY;
-        // If the caller explicitly provides a minimumSize, respect it.
-        // Otherwise fall back to collapsedSize + 50 so the expanded state is
-        // visually distinguishable from the collapsed state.
-        this._expandedMinimumSize =
-            options.minimumSize !== undefined
-                ? options.minimumSize
-                : this._collapsedSize + 50;
         this._lastExpandedSize = (_c = options.initialSize) !== null && _c !== void 0 ? _c : 200;
         if (options.collapsed) {
             this._isCollapsed = true;
             group.element.classList.add('dv-edge-collapsed');
         }
+        this._observeTabStrip();
+    }
+    /**
+     * Watch the group's tab strip so the collapsed size follows the live tab
+     * height. The strip's cross-axis size (height for top/bottom, width for
+     * left/right) is exactly what a collapsed edge group should occupy; a
+     * ResizeObserver keeps them in sync when the tab height changes at runtime.
+     */
+    _observeTabStrip() {
+        const strip = this._group.element.querySelector('.dv-tabs-and-actions-container');
+        if (!strip) {
+            return;
+        }
+        this._tabSizeDisposables.addDisposables(watchElementResize(strip, () => {
+            this._applyMeasuredTabSize(this._orientation === 'vertical'
+                ? strip.offsetHeight
+                : strip.offsetWidth);
+        }));
+    }
+    /**
+     * Apply a freshly-measured tab-strip cross size. When the group is
+     * collapsed this resizes it in the parent splitview so the collapsed strip
+     * always shows the full tab height.
+     */
+    _applyMeasuredTabSize(size) {
+        if (size <= 0 || size === this._measuredTabSize) {
+            return;
+        }
+        this._measuredTabSize = size;
+        if (this._isCollapsed) {
+            this._onDidChange.fire({ size: this.collapsedSize });
+        }
     }
     layout(size, orthogonalSize) {
         // Track the last expanded size so we can restore it after collapsing
@@ -14851,15 +14923,18 @@ class EdgeGroupView {
         this._lastExpandedSize = size;
     }
     /**
-     * Apply new effective collapsed and expanded-minimum sizes after a theme
-     * or gap change. The caller (ShellManager) is responsible for computing
-     * the correct values from the original config and the new gap.
+     * Apply a new base (pre-gap) collapsed size, base minimum size and gap
+     * contribution after a theme or gap change. Base values come from the
+     * original config; the ShellManager owns the gap arithmetic. A live
+     * measured tab size (if any) continues to win over the base collapsed size.
      */
-    updateCollapsedSize(newCollapsedSize, newExpandedMinimumSize) {
-        this._collapsedSize = newCollapsedSize;
-        this._expandedMinimumSize = newExpandedMinimumSize;
+    updateSizing(baseCollapsedSize, baseMinimumSize, gapAdd) {
+        this._baseCollapsedSize = baseCollapsedSize;
+        this._baseMinimumSize = baseMinimumSize;
+        this._gapAdd = gapAdd;
     }
     dispose() {
+        this._tabSizeDisposables.dispose();
         this._onDidChange.dispose();
     }
 }
@@ -14991,15 +15066,6 @@ class MiddleColumnView {
         this._splitview.dispose();
     }
 }
-function adjustedOpts(base, defaultCollapsed, gapAdd) {
-    var _a;
-    const effectiveCollapsed = ((_a = base.collapsedSize) !== null && _a !== void 0 ? _a : defaultCollapsed) + gapAdd;
-    const result = Object.assign(Object.assign({}, base), { collapsedSize: effectiveCollapsed });
-    if (base.minimumSize !== undefined) {
-        result.minimumSize = base.minimumSize + gapAdd;
-    }
-    return result;
-}
 class ShellManager {
     constructor(container, dockviewElement, layoutGrid, gap = 0, defaultCollapsedSize = 35) {
         this._disposables = new CompositeDisposable();
@@ -15025,6 +15091,26 @@ class ShellManager {
         this._middleIndex = 0;
         this._outerSplitview.addView(this._middleColumn, { type: 'distribute' }, 0);
         this._disposables.addDisposables(watchElementResize(this._shellElement, (entry) => {
+            /**
+             * When the shell (or an ancestor) becomes hidden — e.g. a
+             * nested dockview whose `onlyWhenVisible` host panel is
+             * deactivated — the element collapses to (0, 0) or is detached
+             * from the DOM. Propagating that zero size would relayout the
+             * edge-group splitview at 0, clamping the low-priority edge
+             * groups down to their minimum size and destroying the user's
+             * sizing. Skip these cases so sizes are preserved while hidden;
+             * mirrors the guard in the `Resizable` base class. See #1495.
+             *
+             * offsetParent === null is equivalent to display: none on the
+             * element or one of its ancestors.
+             * @see https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/offsetParent
+             */
+            if (!this._shellElement.offsetParent) {
+                return;
+            }
+            if (!isInDocument(this._shellElement)) {
+                return;
+            }
             const width = Math.round(entry.contentRect.width);
             const height = Math.round(entry.contentRect.height);
             if (width === this._currentWidth &&
@@ -15061,7 +15147,7 @@ class ShellManager {
         const isHorizontal = position === 'left' || position === 'right';
         const gapAdd = isHorizontal ? outerGapAdd : innerGapAdd;
         const orientation = isHorizontal ? 'horizontal' : 'vertical';
-        const view = new EdgeGroupView(adjustedOpts(Object.assign({ collapsedSize: this._defaultCollapsedSize }, options), this._defaultCollapsedSize, gapAdd), group, orientation);
+        const view = new EdgeGroupView(Object.assign({ collapsedSize: this._defaultCollapsedSize }, options), group, orientation, gapAdd);
         const initialSize = view.isCollapsed
             ? view.collapsedSize
             : view.lastExpandedSize;
@@ -15128,10 +15214,7 @@ class ShellManager {
         const updateView = (view, baseCfg, gapAdd) => {
             var _a;
             const baseCS = (_a = baseCfg.collapsedSize) !== null && _a !== void 0 ? _a : defaultCollapsedSize;
-            const newCS = baseCS + gapAdd;
-            const baseMS = baseCfg.minimumSize;
-            const newMS = baseMS !== undefined ? baseMS + gapAdd : newCS + 50;
-            view.updateCollapsedSize(newCS, newMS);
+            view.updateSizing(baseCS, baseCfg.minimumSize, gapAdd);
         };
         const topCfg = this._viewConfigs.get('top');
         if (this._topView && topCfg) {
@@ -15382,9 +15465,8 @@ class ShellManager {
         }
     }
     dispose() {
-        var _a;
         this._disposables.dispose();
-        (_a = this._shellElement.parentElement) === null || _a === void 0 ? void 0 : _a.removeChild(this._shellElement);
+        this._shellElement.remove();
     }
 }
 
@@ -15421,9 +15503,9 @@ let _hasWarnedUsingCoreDirectly = false;
  * `process` is undefined.
  */
 function warnIfUsingCoreDirectly() {
+    var _a;
     if (typeof process !== 'undefined' &&
-        process.env &&
-        process.env.NODE_ENV === 'production') {
+        ((_a = process.env) === null || _a === void 0 ? void 0 : _a.NODE_ENV) === 'production') {
         return;
     }
     if (_hasWarnedUsingCoreDirectly || isDockviewPackageLoaded()) {
@@ -15814,7 +15896,7 @@ class DockviewComponent extends BaseGrid {
         // The shell always wraps the dockview element so edge groups can be
         // added at any time via addEdgeGroup() without re-parenting the DOM.
         this.disableResizing = true;
-        container.removeChild(this.element);
+        this.element.remove();
         this._shellManager = new ShellManager(container, this.element, (w, h) => this._layoutFromShell(w, h), (_d = (_c = options.theme) === null || _c === void 0 ? void 0 : _c.gap) !== null && _d !== void 0 ? _d : 0, (_e = options.theme) === null || _e === void 0 ? void 0 : _e.edgeGroupCollapsedSize);
         // The shell wraps the dockview element, so move the popup anchor
         // into the shell so overflow dropdowns in edge groups position correctly
@@ -16036,15 +16118,13 @@ class DockviewComponent extends BaseGrid {
         return _window
             .open()
             .then((popoutContainer) => {
-            var _a, _b, _c;
+            var _a, _b, _c, _d;
             if (_window.isDisposed) {
                 return false;
             }
-            const referenceGroup = (options === null || options === void 0 ? void 0 : options.referenceGroup)
-                ? options.referenceGroup
-                : itemToPopout instanceof DockviewPanel
-                    ? itemToPopout.group
-                    : itemToPopout;
+            const referenceGroup = (_a = options === null || options === void 0 ? void 0 : options.referenceGroup) !== null && _a !== void 0 ? _a : (itemToPopout instanceof DockviewPanel
+                ? itemToPopout.group
+                : itemToPopout);
             const referenceLocation = itemToPopout.api.location.type;
             /**
              * The group that is being added doesn't already exist within the DOM, the most likely occurrence
@@ -16055,7 +16135,7 @@ class DockviewComponent extends BaseGrid {
             if (options === null || options === void 0 ? void 0 : options.overridePopoutGridview) {
                 // Restoring a multi-group window: the anchor group is
                 // already built inside the supplied gridview.
-                group = (_a = options.overridePopoutGroup) !== null && _a !== void 0 ? _a : referenceGroup;
+                group = (_b = options.overridePopoutGroup) !== null && _b !== void 0 ? _b : referenceGroup;
             }
             else if (!isGroupAddedToDom) {
                 group = referenceGroup;
@@ -16086,7 +16166,7 @@ class DockviewComponent extends BaseGrid {
             // a nested splitview layout. The window starts with the single
             // anchor group; further groups arrive via drag-and-drop. On
             // restore a pre-populated gridview is supplied instead.
-            const popoutGridview = (_b = options === null || options === void 0 ? void 0 : options.overridePopoutGridview) !== null && _b !== void 0 ? _b : this.createNestedGridview();
+            const popoutGridview = (_c = options === null || options === void 0 ? void 0 : options.overridePopoutGridview) !== null && _c !== void 0 ? _c : this.createNestedGridview();
             if (!(options === null || options === void 0 ? void 0 : options.overridePopoutGridview)) {
                 popoutGridview.addView(group, Sizing.Distribute, [0]);
             }
@@ -16127,9 +16207,9 @@ class DockviewComponent extends BaseGrid {
                             break;
                         case 'floating':
                         case 'popout':
-                            floatingBox = (_c = this.floatingGroups
+                            floatingBox = (_d = this.floatingGroups
                                 .find((value) => value.group.api.id ===
-                                itemToPopout.api.id)) === null || _c === void 0 ? void 0 : _c.overlay.toJSON();
+                                itemToPopout.api.id)) === null || _d === void 0 ? void 0 : _d.overlay.toJSON();
                             this.removeGroup(referenceGroup);
                             break;
                     }
@@ -16970,7 +17050,7 @@ class DockviewComponent extends BaseGrid {
             const createGroupFromSerializedState = (data) => {
                 const { id, locked, hideHeader, headerPosition, views, activeView, } = data;
                 if (typeof id !== 'string') {
-                    throw new Error('dockview: group id must be of type string');
+                    throw new TypeError('dockview: group id must be of type string');
                 }
                 const group = this.createGroup({
                     id,
@@ -17144,8 +17224,7 @@ class DockviewComponent extends BaseGrid {
                         createdPanels.push(panel);
                     }
                 }
-                for (let i = 0; i < createdPanels.length; i++) {
-                    const panel = createdPanels[i];
+                for (const panel of createdPanels) {
                     const isActive = activeView === panel.id;
                     edgeGroup.model.openPanel(panel, {
                         skipSetActive: !isActive,
@@ -17290,8 +17369,8 @@ class DockviewComponent extends BaseGrid {
         return this.mutation('add', () => this._doAddPanel(options));
     }
     _doAddPanel(options) {
-        var _a, _b;
-        if (this.panels.find((_) => _.id === options.id)) {
+        var _a, _b, _c, _d;
+        if (this.panels.some((_) => _.id === options.id)) {
             throw new Error(`dockview: panel with id ${options.id} already exists`);
         }
         let referenceGroup;
@@ -17310,18 +17389,24 @@ class DockviewComponent extends BaseGrid {
                     : options.position.referencePanel;
                 index = options.position.index;
                 if (!referencePanel) {
-                    throw new Error(`dockview: referencePanel '${options.position.referencePanel}' does not exist`);
+                    const referenceId = typeof options.position.referencePanel === 'string'
+                        ? options.position.referencePanel
+                        : (_a = options.position.referencePanel) === null || _a === void 0 ? void 0 : _a.id;
+                    throw new Error(`dockview: referencePanel '${referenceId}' does not exist`);
                 }
                 referenceGroup = this.findGroup(referencePanel);
             }
             else if (isPanelOptionsWithGroup(options.position)) {
                 referenceGroup =
                     typeof options.position.referenceGroup === 'string'
-                        ? (_a = this._groups.get(options.position.referenceGroup)) === null || _a === void 0 ? void 0 : _a.value
+                        ? (_b = this._groups.get(options.position.referenceGroup)) === null || _b === void 0 ? void 0 : _b.value
                         : options.position.referenceGroup;
                 index = options.position.index;
                 if (!referenceGroup) {
-                    throw new Error(`dockview: referenceGroup '${options.position.referenceGroup}' does not exist`);
+                    const referenceId = typeof options.position.referenceGroup === 'string'
+                        ? options.position.referenceGroup
+                        : (_c = options.position.referenceGroup) === null || _c === void 0 ? void 0 : _c.id;
+                    throw new Error(`dockview: referenceGroup '${referenceId}' does not exist`);
                 }
             }
             else {
@@ -17347,7 +17432,7 @@ class DockviewComponent extends BaseGrid {
         }
         let panel;
         if (referenceGroup) {
-            const target = toTarget(((_b = options.position) === null || _b === void 0 ? void 0 : _b.direction) || 'within');
+            const target = toTarget(((_d = options.position) === null || _d === void 0 ? void 0 : _d.direction) || 'within');
             if (options.floating) {
                 const group = this.createGroup();
                 this._onDidAddGroup.fire(group);
@@ -17429,14 +17514,17 @@ class DockviewComponent extends BaseGrid {
         }
         return panel;
     }
-    removePanel(panel, options = {
-        removeEmptyGroup: true,
-    }) {
-        this.mutation('remove', () => this._doRemovePanel(panel, options));
+    removePanel(panel, options) {
+        this.mutation('remove', () => {
+            var _a;
+            return this._doRemovePanel(panel, {
+                removeEmptyGroup: (_a = options === null || options === void 0 ? void 0 : options.removeEmptyGroup) !== null && _a !== void 0 ? _a : true,
+                skipDispose: options === null || options === void 0 ? void 0 : options.skipDispose,
+                skipSetActiveGroup: options === null || options === void 0 ? void 0 : options.skipSetActiveGroup,
+            });
+        });
     }
-    _doRemovePanel(panel, options = {
-        removeEmptyGroup: true,
-    }) {
+    _doRemovePanel(panel, options) {
         const group = panel.group;
         if (!group) {
             throw new Error(`dockview: cannot remove panel ${panel.id}. it's missing a group.`);
@@ -17462,28 +17550,34 @@ class DockviewComponent extends BaseGrid {
         return this.mutation('add', () => this._doAddGroup(options));
     }
     _doAddGroup(options) {
-        var _a;
+        var _a, _b, _c;
         if (options) {
             let referenceGroup;
             if (isGroupOptionsWithPanel(options)) {
                 const referencePanel = typeof options.referencePanel === 'string'
                     ? this.panels.find((panel) => panel.id === options.referencePanel)
                     : options.referencePanel;
+                const referencePanelId = typeof options.referencePanel === 'string'
+                    ? options.referencePanel
+                    : (_a = options.referencePanel) === null || _a === void 0 ? void 0 : _a.id;
                 if (!referencePanel) {
-                    throw new Error(`dockview: reference panel ${options.referencePanel} does not exist`);
+                    throw new Error(`dockview: reference panel ${referencePanelId} does not exist`);
                 }
                 referenceGroup = this.findGroup(referencePanel);
                 if (!referenceGroup) {
-                    throw new Error(`dockview: reference group for reference panel ${options.referencePanel} does not exist`);
+                    throw new Error(`dockview: reference group for reference panel ${referencePanelId} does not exist`);
                 }
             }
             else if (isGroupOptionsWithGroup(options)) {
                 referenceGroup =
                     typeof options.referenceGroup === 'string'
-                        ? (_a = this._groups.get(options.referenceGroup)) === null || _a === void 0 ? void 0 : _a.value
+                        ? (_b = this._groups.get(options.referenceGroup)) === null || _b === void 0 ? void 0 : _b.value
                         : options.referenceGroup;
                 if (!referenceGroup) {
-                    throw new Error(`dockview: reference group ${options.referenceGroup} does not exist`);
+                    const referenceId = typeof options.referenceGroup === 'string'
+                        ? options.referenceGroup
+                        : (_c = options.referenceGroup) === null || _c === void 0 ? void 0 : _c.id;
+                    throw new Error(`dockview: reference group ${referenceId} does not exist`);
                 }
             }
             else {
@@ -17606,14 +17700,14 @@ class DockviewComponent extends BaseGrid {
             if (this.detachFromNestedWindow(group)) {
                 // The floating window hosts other groups and stays alive —
                 // finalize just this group.
-                if (!(options === null || options === void 0 ? void 0 : options.skipDispose)) {
-                    this.disposeGroupRecord(group);
-                }
-                else {
+                if (options === null || options === void 0 ? void 0 : options.skipDispose) {
                     // Relocation: reset location so the destination root can
                     // re-tag it.
                     group.model.location = { type: 'grid' };
                 }
+                else {
+                    this.disposeGroupRecord(group);
+                }
                 this.activateFallbackGroupIfRemoved(group, options === null || options === void 0 ? void 0 : options.skipActive);
                 return group;
             }
@@ -17634,14 +17728,14 @@ class DockviewComponent extends BaseGrid {
             if (this.detachFromNestedWindow(group)) {
                 // The popout window hosts other groups and stays alive —
                 // finalize just this group.
-                if (!(options === null || options === void 0 ? void 0 : options.skipDispose)) {
-                    this.disposeGroupRecord(group);
-                }
-                else {
+                if (options === null || options === void 0 ? void 0 : options.skipDispose) {
                     // Relocation: reset location so the destination root can
                     // re-tag it.
                     group.model.location = { type: 'grid' };
                 }
+                else {
+                    this.disposeGroupRecord(group);
+                }
                 this.activateFallbackGroupIfRemoved(group, options === null || options === void 0 ? void 0 : options.skipActive);
                 return group;
             }
@@ -17651,7 +17745,7 @@ class DockviewComponent extends BaseGrid {
                     const refGroup = selectedGroup.referenceGroup
                         ? this.getPanel(selectedGroup.referenceGroup)
                         : undefined;
-                    if (refGroup && refGroup.panels.length === 0) {
+                    if ((refGroup === null || refGroup === void 0 ? void 0 : refGroup.panels.length) === 0) {
                         this.removeGroup(refGroup);
                     }
                 }
@@ -18327,9 +18421,7 @@ class DockviewComponent extends BaseGrid {
         return id;
     }
     createGroup(options) {
-        if (!options) {
-            options = {};
-        }
+        options !== null && options !== void 0 ? options : (options = {});
         let id = options === null || options === void 0 ? void 0 : options.id;
         if (id && this._groups.has(options.id)) {
             console.warn(`dockview: Duplicate group id ${options === null || options === void 0 ? void 0 : options.id}. reassigning group id to avoid errors`);
@@ -18525,13 +18617,13 @@ class DockviewComponent extends BaseGrid {
             entry.gridview.margin = gap;
         }
         (_f = this._shellManager) === null || _f === void 0 ? void 0 : _f.updateTheme(gap, (_g = theme.edgeGroupCollapsedSize) !== null && _g !== void 0 ? _g : 35);
-        if (theme.dndOverlayBorder !== undefined) {
-            this.element.style.setProperty('--dv-drag-over-border', theme.dndOverlayBorder);
-            (_h = this._shellManager) === null || _h === void 0 ? void 0 : _h.element.style.setProperty('--dv-drag-over-border', theme.dndOverlayBorder);
+        if (theme.dndOverlayBorder === undefined) {
+            this.element.style.removeProperty('--dv-drag-over-border');
+            (_h = this._shellManager) === null || _h === void 0 ? void 0 : _h.element.style.removeProperty('--dv-drag-over-border');
         }
         else {
-            this.element.style.removeProperty('--dv-drag-over-border');
-            (_j = this._shellManager) === null || _j === void 0 ? void 0 : _j.element.style.removeProperty('--dv-drag-over-border');
+            this.element.style.setProperty('--dv-drag-over-border', theme.dndOverlayBorder);
+            (_j = this._shellManager) === null || _j === void 0 ? void 0 : _j.element.style.setProperty('--dv-drag-over-border', theme.dndOverlayBorder);
         }
         switch (theme.dndOverlayMounting) {
             case 'absolute':
@@ -18773,12 +18865,7 @@ class GridviewComponent extends BaseGrid {
             }
             this._groups.forEach((groupItem) => {
                 const group = groupItem.value;
-                if (group !== panel) {
-                    group.setActive(false);
-                }
-                else {
-                    group.setActive(true);
-                }
+                group.setActive(group === panel);
             });
         }));
         this._groups.set(panel.id, {
@@ -19271,9 +19358,7 @@ class PaneviewComponent extends Resizable {
                 name: options.headerComponent,
             });
         }
-        if (!header) {
-            header = new DefaultHeader();
-        }
+        header !== null && header !== void 0 ? header : (header = new DefaultHeader());
         const view = new PaneFramework({
             id: options.id,
             component: options.component,
@@ -19372,9 +19457,7 @@ class PaneviewComponent extends Resizable {
                             name: data.headerComponent,
                         });
                     }
-                    if (!header) {
-                        header = new DefaultHeader();
-                    }
+                    header !== null && header !== void 0 ? header : (header = new DefaultHeader());
                     const panel = new PaneFramework({
                         id: data.id,
                         component: data.component,