@xh/hoist 78.0.0-SNAPSHOT.1761848630542 → 78.0.0-SNAPSHOT.1762201137843

package/build/types/core/persist/PersistOptions.d.ts

@@ -35,12 +35,12 @@ export interface PersistOptions {
     viewManagerModel?: ViewManagerModel;
     /**
      *  Function returning blob of data to be used for reading state.
-     *  Ignored if `prefKey`, `localStorageKey` or `dashViewModel` are provided.
+     *  Ignored if `prefKey`, `localStorageKey`, `dashViewModel` or 'viewManagerModel' are provided.
      */
     getData?: () => any;
     /**
      * Function to be used to write blob of data representing state.
-     * Ignored if `prefKey`, `localStorageKey` or `dashViewModel` are provided.
+     * Ignored if `prefKey`, `localStorageKey`, `dashViewModel` or 'viewManagerModel' are provided.
      */
     setData?: (data: object) => void;
 }

package/cmp/chart/impl/copyToClipboard.ts

@@ -24,14 +24,14 @@ export function installCopyToClipboard(Highcharts) {
             try {
                 const blobPromise = convertChartToPngAsync(this),
                     clipboardItemInput = new window.ClipboardItem({
-                        // Safari requires an unresolved promise.  See https://bugs.webkit.org/show_bug.cgi?id=222262 for discussion
+                        // Safari requires an unresolved promise. See https://bugs.webkit.org/show_bug.cgi?id=222262 for discussion
                         'image/png': Highcharts.isSafari ? blobPromise : await blobPromise
                     });
                 await window.navigator.clipboard.write([clipboardItemInput]);
                 XH.successToast('Chart copied to clipboard');
             } catch (e) {
                 XH.handleException(e, {showAlert: false, logOnServer: true});
-                XH.dangerToast('Error: Chart could not be copied.  This error has been logged.');
+                XH.dangerToast('Error: Chart could not be copied. This error has been logged.');
             }
         }
     });
@@ -41,14 +41,7 @@ export function installCopyToClipboard(Highcharts) {
 // Implementation
 //------------------
 async function convertChartToPngAsync(chart) {
-    const svg = await new Promise((resolve, reject) =>
-            chart.getSVGForLocalExport(
-                chart.options.exporting,
-                {},
-                () => reject('Cannot fallback to export server'),
-                svg => resolve(svg)
-            )
-        ),
+    const svg = chart.getSVG(),
         svgUrl = svgToDataUrl(svg),
         pngDataUrl = await svgUrlToPngDataUrlAsync(svgUrl),
         ret = await loadBlob(pngDataUrl);
@@ -65,7 +58,7 @@ function memoryCleanup(svgUrl) {
 }
 
 /**
- * Convert dataUri converted to blob
+ * Convert dataUri to blob
  */
 async function loadBlob(dataUrl) {
     const fetched = await fetch(dataUrl);
@@ -84,7 +77,7 @@ function svgToDataUrl(svg) {
     try {
         // Safari requires data URI since it doesn't allow navigation to blob
         // URLs.
-        // foreignObjects dont work well in Blobs in Chrome (#14780).
+        // foreignObjects don't work well in Blobs in Chrome (#14780).
         if (!isWebKitButNotChrome && svg.indexOf('<foreignObject') === -1) {
             return domurl.createObjectURL(
                 new window.Blob([svg], {
@@ -94,7 +87,7 @@ function svgToDataUrl(svg) {
         }
     } catch (e) {}
 
-    // safari, firefox, or svgs with foreignObect returns this
+    // Safari, Firefox, or SVGs with foreignObect returns this
     return 'data:image/svg+xml;charset=UTF-8,' + encodeURIComponent(svg);
 }
 

package/cmp/treemap/TreeMap.ts

@@ -172,7 +172,6 @@ class TreeMapLocalModel extends HoistModel {
             this.prevConfig = cloneDeep(chartCfg);
             this.createChart(config);
         }
-
         this.updateLabelVisibility();
     }
 
@@ -199,9 +198,18 @@ class TreeMapLocalModel extends HoistModel {
         });
     }
 
+    // Reload series data by fully removing and re-adding the series.
+    // When treemap clustering is enabled, `setData()` & `series.update()` does not properly clear old cluster nodes,
+    // causing overlap or stale rendering. Removing and re-adding the series forces a full rebuild
+    // of the layout and clustering state, ensuring the chart is correctly redrawn.
     @logWithDebug
     reloadSeriesData(newData) {
-        this.chart?.series[0].setData(newData, true, false);
+        const {chart} = this;
+        if (!chart) return;
+        const oldSeries = chart.series[0],
+            series = Highcharts.merge(oldSeries.userOptions, {data: newData});
+        oldSeries.remove(false);
+        chart.addSeries(series, true);
     }
 
     startResize = ({width, height}) => {

package/cmp/treemap/TreeMapModel.ts

@@ -465,7 +465,7 @@ export class TreeMapModel extends HoistModel {
     //----------------------
     defaultOnClick = (record, e) => {
         const {gridModel} = this;
-        if (!gridModel) return;
+        if (!gridModel || !record) return;
 
         // Select nodes in grid
         const {selModel} = gridModel;
@@ -477,7 +477,7 @@ export class TreeMapModel extends HoistModel {
     };
 
     defaultOnDoubleClick = record => {
-        if (!this.gridModel?.treeMode || isEmpty(record.children)) return;
+        if (!this.gridModel?.treeMode || isEmpty(record?.children)) return;
         this.toggleNodeExpanded(record.treePath);
     };
 }

package/core/persist/PersistOptions.ts

@@ -59,13 +59,13 @@ export interface PersistOptions {
 
     /**
      *  Function returning blob of data to be used for reading state.
-     *  Ignored if `prefKey`, `localStorageKey` or `dashViewModel` are provided.
+     *  Ignored if `prefKey`, `localStorageKey`, `dashViewModel` or 'viewManagerModel' are provided.
      */
     getData?: () => any;
 
     /**
      * Function to be used to write blob of data representing state.
-     * Ignored if `prefKey`, `localStorageKey` or `dashViewModel` are provided.
+     * Ignored if `prefKey`, `localStorageKey`, `dashViewModel` or 'viewManagerModel' are provided.
      */
     setData?: (data: object) => void;
 }

package/kit/highcharts/index.ts

@@ -9,8 +9,8 @@ import {checkVersion, logError} from '@xh/hoist/utils/js';
 
 export let Highcharts = null;
 
-const MIN_VERSION = '11.1.0';
-const MAX_VERSION = '11.*.*';
+const MIN_VERSION = '12.4.0';
+const MAX_VERSION = '12.*.*';
 
 /**
  * Expose application versions of Highcharts to Hoist.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xh/hoist",
-  "version": "78.0.0-SNAPSHOT.1761848630542",
+  "version": "78.0.0-SNAPSHOT.1762201137843",
   "description": "Hoist add-on for building and deploying React Applications.",
   "repository": "github:xh/hoist-react",
   "homepage": "https://xh.io",