dockview-core 1.13.1 → 1.14.0

package/dist/dockview-core.js

@@ -1,6 +1,6 @@
 /**
  * dockview-core
- * @version 1.13.1
+ * @version 1.14.0
  * @link https://github.com/mathuo/dockview
  * @license MIT
  */
@@ -159,7 +159,7 @@
           this.value = value;
       }
       print() {
-          console.warn(this.value);
+          console.warn('dockview: stacktrace', this.value);
       }
   }
   class Listener {
@@ -224,7 +224,7 @@
                           var _a;
                           // don't check until stack of execution is completed to allow for out-of-order disposals within the same execution block
                           for (const listener of this._listeners) {
-                              console.warn((_a = listener.stacktrace) === null || _a === void 0 ? void 0 : _a.print());
+                              console.warn('dockview: stacktrace', (_a = listener.stacktrace) === null || _a === void 0 ? void 0 : _a.print());
                           }
                       });
                   }
@@ -5774,8 +5774,7 @@
       }
   }
 
-  // TODO find a better way to initialize and avoid needing null checks
-  const NOT_INITIALIZED_MESSAGE = 'DockviewGroupPanelApiImpl not initialized';
+  const NOT_INITIALIZED_MESSAGE = 'dockview: DockviewGroupPanelApiImpl not initialized';
   class DockviewGroupPanelApiImpl extends GridviewPanelApiImpl {
       get location() {
           if (!this._group) {
@@ -5848,14 +5847,14 @@
           }
       }
       initialize(group) {
-          this._group = group;
           /**
-           * TODO: Annoying initialization order caveat
+           * 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) => {
@@ -6009,12 +6008,10 @@
           var _a;
           let _trackGroupActive = (_a = previousGroup === null || previousGroup === void 0 ? void 0 : previousGroup.isActive) !== null && _a !== void 0 ? _a : false; // prevent duplicate events with same state
           this.groupEventsDisposable.value = new CompositeDisposable(this.group.api.onDidVisibilityChange((event) => {
-              if (!event.isVisible && this.isVisible) {
-                  this._onDidVisibilityChange.fire(event);
-              }
-              else if (event.isVisible &&
-                  !this.isVisible &&
-                  this.group.model.isPanelActive(this.panel)) {
+              const hasBecomeHidden = !event.isVisible && this.isVisible;
+              const hasBecomeVisible = event.isVisible && !this.isVisible;
+              const isActivePanel = this.group.model.isPanelActive(this.panel);
+              if (hasBecomeHidden || (hasBecomeVisible && isActivePanel)) {
                   this._onDidVisibilityChange.fire(event);
               }
           }), this.group.api.onDidLocationChange((event) => {
@@ -6100,12 +6097,6 @@
           const didTitleChange = title !== this.title;
           if (didTitleChange) {
               this._title = title;
-              this.view.update({
-                  params: {
-                      params: this._params,
-                      title: this.title,
-                  },
-              });
               this.api._onDidTitleChange.fire({ title });
           }
       }
@@ -6133,10 +6124,7 @@
           }
           // update the view with the updated props
           this.view.update({
-              params: {
-                  params: this._params,
-                  title: this.title,
-              },
+              params: this._params,
           });
       }
       updateParentGroup(group, options) {
@@ -7274,7 +7262,7 @@
               return element.getBoundingClientRect();
           }
           const box = getBox();
-          const groupId = (_b = (_a = options === null || options === void 0 ? void 0 : options.overridePopoutGroup) === null || _a === void 0 ? void 0 : _a.id) !== null && _b !== void 0 ? _b : this.getNextGroupId(); //item.id;
+          const groupId = (_b = (_a = options === null || options === void 0 ? void 0 : options.overridePopoutGroup) === null || _a === void 0 ? void 0 : _a.id) !== null && _b !== void 0 ? _b : this.getNextGroupId();
           if (itemToPopout.api.location.type === 'grid') {
               itemToPopout.api.setVisible(false);
           }
@@ -7390,24 +7378,22 @@
                           });
                       }
                   }
-                  else {
-                      if (this.getPanel(group.id)) {
-                          const removedGroup = this.doRemoveGroup(group, {
-                              skipDispose: true,
-                              skipActive: true,
-                          });
-                          removedGroup.model.renderContainer =
-                              this.overlayRenderContainer;
-                          removedGroup.model.location = { type: 'grid' };
-                          returnedGroup = removedGroup;
-                      }
+                  else if (this.getPanel(group.id)) {
+                      const removedGroup = this.doRemoveGroup(group, {
+                          skipDispose: true,
+                          skipActive: true,
+                      });
+                      removedGroup.model.renderContainer =
+                          this.overlayRenderContainer;
+                      removedGroup.model.location = { type: 'grid' };
+                      returnedGroup = removedGroup;
                   }
               }));
               this._popoutGroups.push(value);
               this.updateWatermark();
           })
               .catch((err) => {
-              console.error(err);
+              console.error('dockview: failed to create popout window', err);
           });
       }
       addFloatingGroup(item, coord, options) {
@@ -7450,7 +7436,7 @@
                       this.doRemoveGroup(item, {
                           skipDispose: true,
                           skipPopoutReturn: true,
-                          skipPopoutAssociated: !!popoutReferenceGroup,
+                          skipPopoutAssociated: false,
                       });
                   }
               }
@@ -7810,7 +7796,6 @@
       clear() {
           const groups = Array.from(this._groups.values()).map((_) => _.value);
           const hasActiveGroup = !!this.activeGroup;
-          !!this.activePanel;
           for (const group of groups) {
               // remove the group will automatically remove the panels
               this.removeGroup(group, { skipActive: true });
@@ -8316,7 +8301,7 @@
           }
           let id = options === null || options === void 0 ? void 0 : options.id;
           if (id && this._groups.has(options.id)) {
-              console.warn(`Duplicate group id ${options === null || options === void 0 ? void 0 : options.id}. reassigning group id to avoid errors`);
+              console.warn(`dockview: Duplicate group id ${options === null || options === void 0 ? void 0 : options.id}. reassigning group id to avoid errors`);
               id = undefined;
           }
           if (!id) {