dockview-core 3.0.2 → 3.1.1

package/dist/dockview-core.amd.noStyle.js

@@ -1,6 +1,6 @@
 /**
  * dockview-core
- * @version 3.0.2
+ * @version 3.1.1
  * @link https://github.com/mathuo/dockview
  * @license MIT
  */
@@ -4742,8 +4742,8 @@ define(['exports'], (function (exports) { 'use strict';
             this.accessor = accessor;
             this.group = group;
             this.content = undefined;
-            this._onChanged = new Emitter();
-            this.onChanged = this._onChanged.event;
+            this._onPointDown = new Emitter();
+            this.onPointerDown = this._onPointDown.event;
             this._onDropped = new Emitter();
             this.onDrop = this._onDropped.event;
             this._onDragStart = new Emitter();
@@ -4773,13 +4773,10 @@ define(['exports'], (function (exports) { 'use strict';
                 },
             });
             this.onWillShowOverlay = this.dropTarget.onWillShowOverlay;
-            this.addDisposables(this._onChanged, this._onDropped, this._onDragStart, dragHandler.onDragStart((event) => {
+            this.addDisposables(this._onPointDown, this._onDropped, this._onDragStart, dragHandler.onDragStart((event) => {
                 this._onDragStart.fire(event);
             }), dragHandler, addDisposableListener(this._element, 'pointerdown', (event) => {
-                if (event.defaultPrevented) {
-                    return;
-                }
-                this._onChanged.fire(event);
+                this._onPointDown.fire(event);
             }), this.dropTarget.onDrop((event) => {
                 this._onDropped.fire(event);
             }), this.dropTarget);
@@ -5057,6 +5054,9 @@ define(['exports'], (function (exports) { 'use strict';
         delete(id) {
             const index = this.tabs.findIndex((tab) => tab.value.panel.id === id);
             const tabToRemove = this.tabs.splice(index, 1)[0];
+            if (!tabToRemove) {
+                throw new Error(`dockview: Tab not found`);
+            }
             const { value, disposable } = tabToRemove;
             disposable.dispose();
             value.dispose();
@@ -5077,7 +5077,10 @@ define(['exports'], (function (exports) { 'use strict';
             tab.setContent(panel.view.tab);
             const disposable = new CompositeDisposable(tab.onDragStart((event) => {
                 this._onTabDragStart.fire({ nativeEvent: event, panel });
-            }), tab.onChanged((event) => {
+            }), tab.onPointerDown((event) => {
+                if (event.defaultPrevented) {
+                    return;
+                }
                 const isFloatingGroupsEnabled = !this.accessor.options.disableFloatingGroups;
                 const isFloatingWithOnePanel = this.group.api.location.type === 'floating' &&
                     this.size === 1;
@@ -5095,12 +5098,12 @@ define(['exports'], (function (exports) { 'use strict';
                     });
                     return;
                 }
-                const isLeftClick = event.button === 0;
-                if (!isLeftClick || event.defaultPrevented) {
-                    return;
-                }
-                if (this.group.activePanel !== panel) {
-                    this.group.model.openPanel(panel);
+                switch (event.button) {
+                    case 0: // left click or touch
+                        if (this.group.activePanel !== panel) {
+                            this.group.model.openPanel(panel);
+                        }
+                        break;
                 }
             }), tab.onDrop((event) => {
                 this._onDrop.fire({
@@ -6050,12 +6053,11 @@ define(['exports'], (function (exports) { 'use strict';
         constructor(id, accessor) {
             super(id, '__dockviewgroup__');
             this.accessor = accessor;
-            this._mutableDisposable = new MutableDisposable();
             this._onDidLocationChange = new Emitter();
             this.onDidLocationChange = this._onDidLocationChange.event;
             this._onDidActivePanelChange = new Emitter();
             this.onDidActivePanelChange = this._onDidActivePanelChange.event;
-            this.addDisposables(this._onDidLocationChange, this._onDidActivePanelChange, this._mutableDisposable);
+            this.addDisposables(this._onDidLocationChange, this._onDidActivePanelChange);
         }
         close() {
             if (!this._group) {
@@ -6113,20 +6115,7 @@ define(['exports'], (function (exports) { 'use strict';
             }
         }
         initialize(group) {
-            /**
-             * TODO: Annoying initialization order caveat, find a better way to initialize and avoid needing null checks
-             *
-             * Due to the order on initialization we know that the model isn't defined until later in the same stack-frame of setup.
-             * By queuing a microtask we can ensure the setup is completed within the same stack-frame, but after everything else has
-             * finished ensuring the `model` is defined.
-             */
             this._group = group;
-            queueMicrotask(() => {
-                this._mutableDisposable.value =
-                    this._group.model.onDidActivePanelChange((event) => {
-                        this._onDidActivePanelChange.fire(event);
-                    });
-            });
         }
     }
 
@@ -6196,6 +6185,9 @@ define(['exports'], (function (exports) { 'use strict';
             }, new DockviewGroupPanelApiImpl(id, accessor));
             this.api.initialize(this); // cannot use 'this' after after 'super' call
             this._model = new DockviewGroupPanelModel(this.element, accessor, id, options, this);
+            this.addDisposables(this.model.onDidActivePanelChange((event) => {
+                this.api._onDidActivePanelChange.fire(event);
+            }));
         }
         focus() {
             if (!this.api.isActive) {
@@ -6543,9 +6535,6 @@ define(['exports'], (function (exports) { 'use strict';
             this.action.appendChild(createCloseButton());
             this._element.appendChild(this._content);
             this._element.appendChild(this.action);
-            this.addDisposables(addDisposableListener(this.action, 'pointerdown', (ev) => {
-                ev.preventDefault();
-            }));
             this.render();
         }
         init(params) {
@@ -7850,6 +7839,9 @@ define(['exports'], (function (exports) { 'use strict';
                 addDisposableWindowListener(_window.window, 'resize', () => {
                     group.layout(_window.window.innerWidth, _window.window.innerHeight);
                 }), overlayRenderContainer, exports.DockviewDisposable.from(() => {
+                    if (this.isDisposed) {
+                        return; // cleanup may run after instance is disposed
+                    }
                     if (isGroupAddedToDom &&
                         this.getPanel(referenceGroup.id)) {
                         this.movingLock(() => moveGroupWithoutDestroying({