dockview-core 7.0.2 → 7.0.4

package/dist/esm/api/dockviewGroupPanelApi.js

@@ -72,7 +72,7 @@ export class DockviewGroupPanelApiImpl extends GridviewPanelApiImpl {
     getWindow() {
         return this.location.type === 'popout'
             ? this.location.getWindow()
-            : window;
+            : globalThis.window;
     }
     setHeaderPosition(position) {
         if (!this._group) {

package/dist/esm/dnd/dropTargetAnchorContainer.js

@@ -19,9 +19,8 @@ export 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;
             },

package/dist/esm/dnd/pointer/longPress.js

@@ -33,7 +33,7 @@ export 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();

package/dist/esm/dnd/pointer/pointerDragController.js

@@ -14,9 +14,8 @@ import { CompositeDisposable } from '../../lifecycle';
  */
 export 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() {
@@ -81,21 +80,24 @@ export 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);

package/dist/esm/dnd/pointer/pointerDragSource.js

@@ -82,7 +82,9 @@ export 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;
@@ -94,7 +96,7 @@ export 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;

package/dist/esm/dockview/components/panel/content.js

@@ -46,7 +46,7 @@ export 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');
@@ -113,7 +113,7 @@ export 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;
@@ -131,7 +131,7 @@ export 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,
@@ -159,11 +159,11 @@ export 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;

package/dist/esm/dockview/components/popupService.js

@@ -10,7 +10,7 @@ function isCoarsePrimaryInput(win) {
     return coarse && !fine;
 }
 export class PopupService extends CompositeDisposable {
-    constructor(root, win = window) {
+    constructor(root, win = globalThis.window) {
         super();
         this._active = null;
         this._activeDisposable = new MutableDisposable();

package/dist/esm/dockview/components/tab/tab.js

@@ -55,7 +55,7 @@ export 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') {
@@ -156,7 +156,7 @@ export 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);

package/dist/esm/dockview/components/titlebar/floatingTitleBar.d.ts

@@ -1,5 +1,4 @@
 import { DockviewComponent } from '../../dockviewComponent';
-import { Event } from '../../../events';
 import { CompositeDisposable } from '../../../lifecycle';
 import { DockviewGroupPanel } from '../../dockviewGroupPanel';
 /**
@@ -20,7 +19,7 @@ export declare class FloatingTitleBar extends CompositeDisposable {
     private readonly dragSource;
     private _group;
     private readonly _onDragStart;
-    readonly onDragStart: Event<PointerEvent | DragEvent>;
+    readonly onDragStart: import("../../../events").Event<PointerEvent | DragEvent>;
     get element(): HTMLElement;
     /** The window's current anchor group — the one this bar drags/activates. */
     get group(): DockviewGroupPanel;

package/dist/esm/dockview/components/titlebar/groupDragSource.d.ts

@@ -1,5 +1,4 @@
 import { DockviewComponent } from '../../dockviewComponent';
-import { Event } from '../../../events';
 import { CompositeDisposable } from '../../../lifecycle';
 import { DockviewGroupPanel } from '../../dockviewGroupPanel';
 declare const FLOATING_REDOCK_INITIATION_DELAY_MS = 500;
@@ -42,7 +41,7 @@ export declare class GroupDragSource extends CompositeDisposable {
     private readonly pointerDragSource;
     private readonly panelTransfer;
     private readonly _onDragStart;
-    readonly onDragStart: Event<PointerEvent | DragEvent>;
+    readonly onDragStart: import("../../../events").Event<PointerEvent | DragEvent>;
     private readonly isFloatingMoveHandle;
     private get group();
     constructor(options: GroupDragSourceOptions);

package/dist/esm/dockview/components/titlebar/groupDragSource.js

@@ -42,7 +42,7 @@ export 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;

package/dist/esm/dockview/components/titlebar/tabGroupIndicator.js

@@ -142,8 +142,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();
@@ -240,7 +239,10 @@ export 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';
@@ -249,9 +251,6 @@ export 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) {

package/dist/esm/dockview/components/titlebar/tabGroups.d.ts

@@ -27,9 +27,17 @@ export interface TabGroupManagerCallbacks {
 }
 interface ChipRendererEntry {
     chip: ITabGroupChipRenderer;
-    /** Created by the manager so it can be toggled live on strategy changes. */
-    html5DragSource: IDragSource;
-    pointerDragSource: IDragSource;
+    /**
+     * Created by the manager so it can be toggled live on strategy changes.
+     * Undefined between an in-flight drag being torn down by
+     * `disposeChipDrag` and the following `update()` re-arming it — the chip
+     * element outlives a within-group move even though its drag sources do
+     * not. (#1410)
+     */
+    html5DragSource: IDragSource | undefined;
+    pointerDragSource: IDragSource | undefined;
+    /** Disposes the two drag sources + their shared dragend listener. */
+    dragSourcesDisposable: IDisposable | undefined;
     disposable: IDisposable;
     dropTarget: Droptarget;
 }
@@ -70,14 +78,28 @@ export declare 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: string): void;
     /** Cloned chip rect used as the pointer follow-finger ghost. */
     private _buildChipGhostElement;
     disposeAll(): void;
     private _ensureIndicator;
+    /**
+     * 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)
+     */
+    private _createChipDragSources;
     private _ensureChipForGroup;
     private _positionChipForGroup;
     private _updateTabGroupClasses;

package/dist/esm/dockview/components/titlebar/tabGroups.js

@@ -1,6 +1,6 @@
 import { toggleClass } from '../../../dom';
 import { addDisposableListener } from '../../../events';
-import { LocalSelectionTransfer, PanelTransfer, } from '../../../dnd/dataTransfer';
+import { getPanelData, LocalSelectionTransfer, PanelTransfer, } from '../../../dnd/dataTransfer';
 import { html5Backend, pointerBackend, } from '../../../dnd/backend';
 import { LongPressDetector } from '../../../dnd/pointer/longPress';
 import { CompositeDisposable, } from '../../../lifecycle';
@@ -8,7 +8,6 @@ import { resolveDndCapabilities } from '../../dndCapabilities';
 import { applyTabGroupAccent } from '../../tabGroupAccent';
 import { TabGroupChip } from './tabGroupChip';
 import { Droptarget } from '../../../dnd/droptarget';
-import { getPanelData } from '../../../dnd/dataTransfer';
 import { NoneTabGroupIndicator, WrapTabGroupIndicator, } from './tabGroupIndicator';
 const EMPTY_MAP = new Map();
 export class TabGroupManager {
@@ -38,6 +37,7 @@ export 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
@@ -52,6 +52,7 @@ export 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);
             }
@@ -180,12 +181,13 @@ export 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);
         }
     }
     /**
@@ -194,19 +196,28 @@ export 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) {
@@ -219,7 +230,7 @@ export 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) {
@@ -229,12 +240,13 @@ export 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)
@@ -242,28 +254,24 @@ export 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();
@@ -315,9 +323,12 @@ export 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,
@@ -332,6 +343,41 @@ export 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;
@@ -345,17 +391,19 @@ export 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) {
@@ -388,7 +436,7 @@ export 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;
@@ -404,8 +452,9 @@ export 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,
         });

package/dist/esm/dockview/components/titlebar/tabs.js

@@ -1,4 +1,4 @@
-import { getPanelData, } from '../../../dnd/dataTransfer';
+import { getPanelData } from '../../../dnd/dataTransfer';
 import { addClasses, isChildEntirelyVisibleWithinParent, OverflowObserver, removeClasses, toggleClass, } from '../../../dom';
 import { addDisposableListener, Emitter } from '../../../events';
 import { CompositeDisposable, Disposable, MutableDisposable, } from '../../../lifecycle';
@@ -445,12 +445,10 @@ export 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
@@ -562,18 +560,13 @@ export class Tabs extends CompositeDisposable {
                 });
                 return;
             }
-            switch (event.button) {
-                case 0:
-                    if (this.group.api.location.type === 'edge') {
-                        // All tab interaction for edge groups is handled by
-                        // onTabClick to avoid race conditions with active panel state
-                    }
-                    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;
@@ -596,12 +589,12 @@ export 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 {
@@ -851,7 +844,7 @@ export 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 -

package/dist/esm/dockview/components/titlebar/voidContainer.js

@@ -52,7 +52,7 @@ export 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');

package/dist/esm/dockview/dndCapabilities.js

@@ -24,13 +24,13 @@ export function resolveDndCapabilities(options) {
     }
 }
 function isCoarsePrimaryInput() {
-    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;
 }