@malloy-publisher/server 0.2.6 → 0.3.0

package/dist/runtime/publisher.js

@@ -251,27 +251,73 @@
             var pad = parseFloat(bodyStyle.paddingBottom) || 0;
             return Math.ceil(maxBottom + scrollTop + pad);
          }
-         function postSize() {
+         // Smallest SHRINK worth reporting. measureContentHeight already ceils,
+         // so nothing sub-pixel reaches here; what this absorbs is a 1-2px
+         // rounding flip-flop between two layout passes, which the host cannot
+         // act on without the resize itself becoming a layout change inside this
+         // frame — the feedback loop.
+         //
+         // Applied to shrinks ONLY, deliberately. A frame a few pixels too tall
+         // shows dead space; a few pixels too short CLIPS, and the frame has no
+         // internal scrollbar. The asymmetry also stops the two sides' bands
+         // compounding: the host runs its own epsilon against ITS height, which
+         // its MIN clamp guarantees can differ from lastHeight here, so two
+         // symmetric 8px bands leave a dead zone wider than either. Growing at
+         // 1px precision keeps the host's band the only one that applies.
+         var RESIZE_EPSILON = 8;
+         var resizePending = false;
+
+         function emitSize() {
             var h = measureContentHeight();
-            if (h !== lastHeight) {
-               lastHeight = h;
-               try {
-                  window.parent.postMessage(
-                     { type: "publisher:resize", height: h },
-                     "*",
-                  );
-               } catch (_e) {
-                  /* ignore */
-               }
+            if (h < lastHeight && lastHeight - h < RESIZE_EPSILON) return;
+            if (h === lastHeight) return;
+            lastHeight = h;
+            try {
+               window.parent.postMessage(
+                  { type: "publisher:resize", height: h },
+                  "*",
+               );
+            } catch (_e) {
+               /* ignore */
             }
          }
+
+         // Coalesce a burst into one message per frame.
+         //
+         // ResizeObserver on documentElement fires on EVERY layout change, so a
+         // dashboard whose tiles resolve one by one used to post a height per
+         // tile. The host resized on each, which is what made an embedded app
+         // visibly jitter while it loaded. Same `pending`-flag idiom as the SSE
+         // reload debounce below; rAF rather than a timeout because the next
+         // paint is exactly when a coalesced layout is worth measuring.
+         //
+         // Trailing edge, not leading: the last measurement in a burst is the
+         // settled one, and reporting the first would send a stale height.
+         function postSize() {
+            if (resizePending) return;
+            resizePending = true;
+            var schedule =
+               typeof requestAnimationFrame === "function"
+                  ? requestAnimationFrame
+                  : function (fn) {
+                       setTimeout(fn, 16);
+                    };
+            schedule(function () {
+               resizePending = false;
+               emitSize();
+            });
+         }
          // Initial + observe content changes
+         // The first report goes out synchronously: the host shows a
+         // MIN_EMBED_HEIGHT placeholder until one arrives, so deferring it by a
+         // frame would leave a collapsed frame for no benefit. Only the bursts
+         // that follow need coalescing.
          if (document.readyState === "loading") {
-            document.addEventListener("DOMContentLoaded", postSize);
+            document.addEventListener("DOMContentLoaded", emitSize);
          } else {
-            postSize();
+            emitSize();
          }
-         window.addEventListener("load", postSize);
+         window.addEventListener("load", emitSize);
          if (typeof ResizeObserver !== "undefined") {
             var ro = new ResizeObserver(postSize);
             // Observe documentElement so we catch any layout change