dockview-core 3.0.1 → 3.1.0

package/dist/dockview-core.noStyle.js

@@ -1,6 +1,6 @@
 /**
  * dockview-core
- * @version 3.0.1
+ * @version 3.1.0
  * @link https://github.com/mathuo/dockview
  * @license MIT
  */
@@ -854,7 +854,7 @@
             this._margin = value;
         }
         constructor(container, options) {
-            var _a;
+            var _a, _b;
             this.container = container;
             this.viewItems = [];
             this.sashes = [];
@@ -952,9 +952,9 @@
                 //
                 return delta;
             };
-            this._orientation = options.orientation;
+            this._orientation = (_a = options.orientation) !== null && _a !== void 0 ? _a : exports.Orientation.VERTICAL;
             this.element = this.createContainer();
-            this.margin = (_a = options.margin) !== null && _a !== void 0 ? _a : 0;
+            this.margin = (_b = options.margin) !== null && _b !== void 0 ? _b : 0;
             this.proportionalLayout =
                 options.proportionalLayout === undefined
                     ? true
@@ -2773,9 +2773,9 @@
         set locked(value) {
             this.gridview.locked = value;
         }
-        constructor(parentElement, options) {
+        constructor(container, options) {
             var _a;
-            super(parentElement, options.disableAutoResizing);
+            super(document.createElement('div'), options.disableAutoResizing);
             this._id = nextLayoutId$1.next();
             this._groups = new Map();
             this._onDidRemove = new Emitter();
@@ -2794,6 +2794,8 @@
             this.element.style.width = '100%';
             this._classNames = new Classnames(this.element);
             this._classNames.setClassNames((_a = options.className) !== null && _a !== void 0 ? _a : '');
+            // the container is owned by the third-party, do not modify/delete it
+            container.appendChild(this.element);
             this.gridview = new Gridview(!!options.proportionalLayout, options.styles, options.orientation, options.locked, options.margin);
             this.gridview.locked = !!options.locked;
             this.element.appendChild(this.gridview.element);
@@ -5059,6 +5061,9 @@
         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();
@@ -5080,6 +5085,9 @@
             const disposable = new CompositeDisposable(tab.onDragStart((event) => {
                 this._onTabDragStart.fire({ nativeEvent: event, panel });
             }), tab.onChanged((event) => {
+                if (event.defaultPrevented) {
+                    return;
+                }
                 const isFloatingGroupsEnabled = !this.accessor.options.disableFloatingGroups;
                 const isFloatingWithOnePanel = this.group.api.location.type === 'floating' &&
                     this.size === 1;
@@ -5097,12 +5105,15 @@
                     });
                     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;
+                    case 1: // middle click
+                        panel.api.close();
+                        break;
                 }
             }), tab.onDrop((event) => {
                 this._onDrop.fire({
@@ -6052,12 +6063,11 @@
         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) {
@@ -6115,20 +6125,7 @@
             }
         }
         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);
-                    });
-            });
         }
     }
 
@@ -6198,6 +6195,9 @@
             }, 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) {
@@ -6545,9 +6545,6 @@
             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) {
@@ -7513,9 +7510,9 @@
         get floatingGroups() {
             return this._floatingGroups;
         }
-        constructor(parentElement, options) {
+        constructor(container, options) {
             var _a;
-            super(parentElement, {
+            super(container, {
                 proportionalLayout: true,
                 orientation: exports.Orientation.HORIZONTAL,
                 styles: options.hideBorders
@@ -7852,6 +7849,9 @@
                 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({
@@ -7871,6 +7871,16 @@
                         group.model.renderContainer =
                             this.overlayRenderContainer;
                         returnedGroup = group;
+                        const alreadyRemoved = !this._popoutGroups.find((p) => p.popoutGroup === group);
+                        if (alreadyRemoved) {
+                            /**
+                             * If this popout group was explicitly removed then we shouldn't run the additional
+                             * steps. To tell if the running of this disposable is the result of this popout group
+                             * being explicitly removed we can check if this popout group is still referenced in
+                             * the `this._popoutGroups` list.
+                             */
+                            return;
+                        }
                         if (floatingBox) {
                             this.addFloatingGroup(group, {
                                 height: floatingBox.height,
@@ -9047,9 +9057,9 @@
         set deserializer(value) {
             this._deserializer = value;
         }
-        constructor(parentElement, options) {
+        constructor(container, options) {
             var _a;
-            super(parentElement, {
+            super(container, {
                 proportionalLayout: (_a = options.proportionalLayout) !== null && _a !== void 0 ? _a : true,
                 orientation: options.orientation,
                 styles: options.hideBorders
@@ -9338,9 +9348,9 @@
                 ? this.splitview.size
                 : this.splitview.orthogonalSize;
         }
-        constructor(parentElement, options) {
+        constructor(container, options) {
             var _a;
-            super(parentElement, options.disableAutoResizing);
+            super(document.createElement('div'), options.disableAutoResizing);
             this._splitviewChangeDisposable = new MutableDisposable();
             this._panels = new Map();
             this._onDidLayoutfromJSON = new Emitter();
@@ -9351,8 +9361,12 @@
             this.onDidRemoveView = this._onDidRemoveView.event;
             this._onDidLayoutChange = new Emitter();
             this.onDidLayoutChange = this._onDidLayoutChange.event;
+            this.element.style.height = '100%';
+            this.element.style.width = '100%';
             this._classNames = new Classnames(this.element);
             this._classNames.setClassNames((_a = options.className) !== null && _a !== void 0 ? _a : '');
+            // the container is owned by the third-party, do not modify/delete it
+            container.appendChild(this.element);
             this._options = options;
             this.splitview = new Splitview(this.element, options);
             this.addDisposables(this._onDidAddView, this._onDidLayoutfromJSON, this._onDidRemoveView, this._onDidLayoutChange);
@@ -9662,9 +9676,9 @@
         get options() {
             return this._options;
         }
-        constructor(parentElement, options) {
+        constructor(container, options) {
             var _a;
-            super(parentElement, options.disableAutoResizing);
+            super(document.createElement('div'), options.disableAutoResizing);
             this._id = nextLayoutId.next();
             this._disposable = new MutableDisposable();
             this._viewDisposables = new Map();
@@ -9680,9 +9694,13 @@
             this.onDidRemoveView = this._onDidRemoveView.event;
             this._onUnhandledDragOverEvent = new Emitter();
             this.onUnhandledDragOverEvent = this._onUnhandledDragOverEvent.event;
+            this.element.style.height = '100%';
+            this.element.style.width = '100%';
             this.addDisposables(this._onDidLayoutChange, this._onDidLayoutfromJSON, this._onDidDrop, this._onDidAddView, this._onDidRemoveView, this._onUnhandledDragOverEvent);
             this._classNames = new Classnames(this.element);
             this._classNames.setClassNames((_a = options.className) !== null && _a !== void 0 ? _a : '');
+            // the container is owned by the third-party, do not modify/delete it
+            container.appendChild(this.element);
             this._options = options;
             this.paneview = new Paneview(this.element, {
                 // only allow paneview in the vertical orientation for now