flexdesk 0.6.2 → 0.6.3

package/dist/charts.js

@@ -39,10 +39,10 @@ import {
   setPlotlySource,
   uid,
   updateChart
-} from "./chunk-XYUZF5CD.js";
+} from "./chunk-WAVLVXYV.js";
 import {
   createRafResizeObserver
-} from "./chunk-CPYBQN3Q.js";
+} from "./chunk-WLASPRGP.js";
 import {
   ManagedWindow
 } from "./chunk-6JSOVNID.js";

package/dist/{chunk-XYUZF5CD.js → chunk-WAVLVXYV.js}

@@ -1,7 +1,7 @@
 import {
   DataTable,
   createRafResizeObserver
-} from "./chunk-CPYBQN3Q.js";
+} from "./chunk-WLASPRGP.js";
 import {
   ManagedWindow
 } from "./chunk-6JSOVNID.js";
@@ -2370,4 +2370,4 @@ export {
   openPlotPopoutWindow,
   openRawTracesWindow
 };
-//# sourceMappingURL=chunk-XYUZF5CD.js.map
+//# sourceMappingURL=chunk-WAVLVXYV.js.map

package/dist/{chunk-CPYBQN3Q.js → chunk-WLASPRGP.js}

@@ -1073,7 +1073,10 @@ var DataTable = class {
       });
     }
     if (this._headerTableEl && this._tableEl) {
-      requestAnimationFrame(() => this._syncHeaderWidths());
+      requestAnimationFrame(() => {
+        this._syncHeaderWidths();
+        this._virtRemeasure();
+      });
       if (this._resizeObserver) {
         try {
           this._resizeObserver.disconnect();
@@ -2411,26 +2414,51 @@ var DataTable = class {
     }
     return lo;
   }
-  /** Measure one real row. Client pixels, so the zoom is divided back out —
-   *  `_measureNaturalWidths` explains why — and NOT `offsetHeight`, which is
-   *  rounded: 21 for a 21.19px row is a 1,100px error over 6,000 rows. */
+  /**
+   * Measure the rows that are DRAWN — the median of up to 64 of them. Client
+   * pixels, so the zoom is divided back out (`_measureNaturalWidths` explains
+   * why), and NOT `offsetHeight`, which is rounded: 21 for a 21.19px row is a
+   * 1,100px error over 6,000 rows.
+   *
+   * 0.6.3: IT USED TO MEASURE ONE ROW, ONCE, AT RENDER, and trust it for the
+   * life of the table. A row measured at a bad moment — a box not yet laid
+   * out at its width, a font not yet loaded — set every row's height: a
+   * Tables grid drawing 22.8px rows was found holding a 134px estimate, so the
+   * window took a 650px viewport for five rows, drew rows 1–35 and left the
+   * rest of the box blank spacer while the reader scrolled. Now it is asked
+   * again on every window update (`_virtUpdate`), on a resize and once the
+   * column widths are fixed, and corrects itself the first time real rows
+   * are on screen. The first measure is exact; after that a change has to be
+   * larger than half a pixel, so rows of mixed heights cannot make the
+   * offsets move under the reader on every scroll.
+   *
+   * @returns {boolean} whether the heights changed (the offsets are recomputed).
+   */
   _virtMeasure() {
     const v = this._virt;
     if (!v) return false;
+    const zoom = this._tableEl?.currentCSSZoom || 1;
+    const heights = [];
     for (const tr of v.map.values()) {
       if (!tr.isConnected || tr.hidden) continue;
-      const zoom = this._tableEl?.currentCSSZoom || 1;
-      const h = tr.getBoundingClientRect().height / zoom;
-      if (!(h > 0)) return false;
-      if (Math.abs(h - v.base) > 0.01) {
-        v.base = h;
-        this._virtBase = h;
-        this._virtComputeOffsets();
-        return true;
-      }
-      return false;
-    }
-    return false;
+      const h2 = tr.getBoundingClientRect().height / zoom;
+      if (h2 > 0) heights.push(h2);
+      if (heights.length >= 64) break;
+    }
+    if (!heights.length) return false;
+    heights.sort((a, b) => a - b);
+    const h = heights[heights.length >> 1];
+    const tolerance = v.measured ? 0.5 : 0.01;
+    v.measured = true;
+    if (Math.abs(h - v.base) <= tolerance) return false;
+    v.base = h;
+    this._virtBase = h;
+    this._virtComputeOffsets();
+    return true;
+  }
+  /** Measure again, and redraw the window when the heights moved. */
+  _virtRemeasure() {
+    if (this._virtMeasure()) this._virtUpdate({ force: true });
   }
   /**
    * Slide the window to wherever the scroll box now is.
@@ -2449,9 +2477,10 @@ var DataTable = class {
    * @param {number}  [o.viewport] the viewport height, likewise.
    * @returns {HTMLElement[]} the rows added.
    */
-  _virtUpdate({ force = false, notify = true, top = null, viewport = null } = {}) {
+  _virtUpdate({ force = false, notify = true, top = null, viewport = null, measure = true } = {}) {
     const v = this._virt;
     if (!v || !v.offsets) return [];
+    if (measure && this._virtMeasure()) force = true;
     const n = v.rows.length;
     const wrap = this._tableWrapEl;
     const view = viewport || wrap?.clientHeight || VIRTUAL_FALLBACK_VIEWPORT_PX;
@@ -2479,6 +2508,15 @@ var DataTable = class {
           console.error("[DataTable] onRowsRendered threw", err);
         }
       }
+      if (measure && added.length && this._virtMeasure()) {
+        return added.concat(this._virtUpdate({
+          force: true,
+          notify,
+          top,
+          viewport,
+          measure: false
+        }));
+      }
     }
     return added;
   }
@@ -3997,4 +4035,4 @@ export {
   resolvePaging,
   DataTable
 };
-//# sourceMappingURL=chunk-CPYBQN3Q.js.map
+//# sourceMappingURL=chunk-WLASPRGP.js.map