@shival99/z-ui 2.0.81 → 2.0.82

package/fesm2022/shival99-z-ui-components-z-table.mjs

@@ -6701,8 +6701,7 @@ class ZTableComponent {
         },
     ];
     _bulkBarExitDuration = 140;
-    /** Stable identity key per virtual index (row/group id, not array index) so cached measured
-     *  heights follow row identity — prevents height jitter when scrolling fast. */
+    /** Stable identity key per virtual index (row/group id, not array index) — prevents height jitter on fast scroll. */
     _virtualItemKeys = computed(() => {
         if (!this.isVirtual()) {
             return [];
@@ -6717,9 +6716,7 @@ class ZTableComponent {
         }
         return Array.from({ length: this._virtualGroupCount() }, (_, index) => index);
     }, ...(ngDevMode ? [{ debugName: "_virtualItemKeys" }] : []));
-    /** Estimate height learned from real rendered rows (dynamicSize); `virtual.size` is the fallback
-     *  until a row is measured. Lets not-yet-measured rows estimate close to reality, reducing layout
-     *  shift. Plain field (non-reactive) so updating it won't rebuild the virtualizer. */
+    /** Estimate height learned from rendered rows (dynamicSize); falls back to `virtual.size`. Reduces layout shift. */
     _dynamicEstimateHeight = null;
     /** Virtualizer instance — operates on groups rather than individual rows */
     virtualizer = injectVirtualizer(() => {
@@ -6760,7 +6757,6 @@ class ZTableComponent {
                 elementScroll(offset, options, instance);
             },
             measureElement: (element) => element.getBoundingClientRect().height,
-            useAnimationFrameWithResizeObserver: true,
             overscan: virtualConfig.overscan ?? Z_DEFAULT_VIRTUAL_OVERSCAN,
             initialOffset: virtualConfig.initialOffset ?? 0,
             ...pickDefined(virtualConfig, [
@@ -6773,33 +6769,22 @@ class ZTableComponent {
             ]),
         };
     });
-    _measureVirtualItems = effect(onCleanup => {
+    _measureVirtualItems = effect(() => {
         if (!this.dynamicSize()) {
             return;
         }
+        // Measure synchronously (no rAF) so sizes are corrected before paint — avoids first-scroll jitter.
         const elements = this.virtualRowElements();
-        const view = this._document.defaultView;
-        if (!view) {
-            return;
-        }
-        // Only learn the estimate until it's locked; afterwards just measure.
         const needsEstimate = this._dynamicEstimateHeight === null;
-        // Batch measurements into the next frame to avoid repositioning multiple times per scroll frame.
-        const frameId = view.requestAnimationFrame(() => {
-            // measureElement() does its own getBoundingClientRect internally — do NOT read the rect again
-            // here, the second read would force an extra reflow per row and tank scroll performance.
-            for (const el of elements) {
-                this.virtualizer.measureElement(el.nativeElement);
-            }
-            if (needsEstimate) {
-                this._learnDynamicEstimate();
-            }
-        });
-        onCleanup(() => view.cancelAnimationFrame(frameId));
+        for (const el of elements) {
+            this.virtualizer.measureElement(el.nativeElement);
+        }
+        // Defer learning to a microtask — it calls virtualizer.measure() (writes signals).
+        if (needsEstimate) {
+            queueMicrotask(() => this._learnDynamicEstimate());
+        }
     }, ...(ngDevMode ? [{ debugName: "_measureVirtualItems" }] : []));
-    /** Learn estimate height from already-measured rows. Reads sizes from the virtualizer's measurement
-     *  cache (no DOM reads / reflow). Locks ONCE so the estimate doesn't jump every scroll frame; each
-     *  row is still measured exactly on render. Re-flows via virtualizer.measure(). */
+    /** Learn estimate height from measured rows (reads the measurement cache, no reflow). Locks once. */
     _learnDynamicEstimate() {
         if (this._dynamicEstimateHeight !== null) {
             return;
@@ -6808,7 +6793,7 @@ class ZTableComponent {
         let measuredSum = 0;
         let measuredCount = 0;
         for (const item of this.virtualizer.getVirtualItems()) {
-            // Skip items still sitting at the default estimate — only real measured rows inform the average.
+            // Skip items still at the default estimate — only real measured rows inform the average.
             if (item.size > 0 && item.size !== rowHeight) {
                 measuredSum += item.size;
                 measuredCount++;