@fundar/data-chart-telling 0.0.38 → 0.0.39

package/dist/plots/utils/labelOverflow.svelte.js

@@ -37,12 +37,21 @@ export function watchLabelOverflow(args) {
             catch {
                 return;
             }
-            const overflow = {
-                left: Math.ceil(Math.max(0, bounds.left - box.x)),
-                right: Math.ceil(Math.max(0, box.x + box.width - bounds.right)),
-                top: Math.ceil(Math.max(0, bounds.top - box.y)),
-                bottom: Math.ceil(Math.max(0, box.y + box.height - bounds.bottom)),
-            };
+            // An empty group (no labels currently rendered, e.g. every series is
+            // scrubbed/filtered out of view) reports getBBox() as (0,0,0,0) — the
+            // SVG origin, not "no content". Reading that literally as a box sitting
+            // at (0,0) would report overflow proportional to the *current* margins
+            // themselves, which grows them, which remounts this group against the
+            // new, larger bounds, which reports overflow again — an unbounded loop
+            // with nothing rendered to justify it.
+            const overflow = box.width === 0 && box.height === 0
+                ? NO_OVERFLOW
+                : {
+                    left: Math.ceil(Math.max(0, bounds.left - box.x)),
+                    right: Math.ceil(Math.max(0, box.x + box.width - bounds.right)),
+                    top: Math.ceil(Math.max(0, bounds.top - box.y)),
+                    bottom: Math.ceil(Math.max(0, box.y + box.height - bounds.bottom))
+                };
             attempt += 1;
             const stable = lastReading != null &&
                 lastReading.left === overflow.left &&
@@ -86,7 +95,7 @@ export function createLabelMarginTracker(margins) {
             left: Math.max(measured.left, overflow.left),
             right: Math.max(measured.right, overflow.right),
             top: Math.max(measured.top, overflow.top),
-            bottom: Math.max(measured.bottom, overflow.bottom),
+            bottom: Math.max(measured.bottom, overflow.bottom)
         };
         if (next.left !== measured.left ||
             next.right !== measured.right ||
@@ -126,11 +135,15 @@ export function createLabelMarginTracker(margins) {
     // must track.
     const plotProps = $derived.by(() => ({
         margins: resolvedMargins,
-        marginRemountKey: JSON.stringify(measured),
+        marginRemountKey: JSON.stringify(measured)
     }));
     return {
         report,
-        get resolvedMargins() { return resolvedMargins; },
-        get plotProps() { return plotProps; },
+        get resolvedMargins() {
+            return resolvedMargins;
+        },
+        get plotProps() {
+            return plotProps;
+        }
     };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fundar/data-chart-telling",
-  "version": "0.0.38",
+  "version": "0.0.39",
   "type": "module",
   "files": [
     "dist"