@opendata-ai/openchart-engine 8.0.0-rc.25 → 8.0.0-rc.26

package/dist/index.js

@@ -8560,6 +8560,11 @@ function computePieLabels(marks, _chartArea, density = "auto", _textFill = "#333
       width: textWidth,
       height: textHeight,
       priority: "data",
+      // Index into the ORIGINAL marks array, not `targetMarks`. Density
+      // filtering drops slices and resolveCollisions re-sorts by priority, so
+      // the returned labels line up with neither. Callers assign by this index
+      // instead of zipping positionally.
+      index: marks.indexOf(mark),
       style: {
         fontFamily: "system-ui, -apple-system, sans-serif",
         fontSize: LABEL_FONT_SIZE5,
@@ -8581,14 +8586,16 @@ function computePieLabels(marks, _chartArea, density = "auto", _textFill = "#333
       x: c.anchorX,
       y: c.anchorY,
       style: c.style,
-      visible: true
+      visible: true,
+      ...c.index !== void 0 ? { index: c.index } : {}
     }));
   } else {
     resolved = resolveCollisions5(candidates);
   }
-  for (let i = 0; i < resolved.length && i < targetMarks.length; i++) {
+  for (let i = 0; i < resolved.length; i++) {
     const label = resolved[i];
-    const mark = targetMarks[i];
+    const mark = label.index !== void 0 ? marks[label.index] : void 0;
+    if (!mark) continue;
     if (label.visible) {
       label.connector = {
         from: { x: label.x, y: label.y },
@@ -8606,16 +8613,16 @@ function computePieLabels(marks, _chartArea, density = "auto", _textFill = "#333
 var pieRenderer = (spec, scales, chartArea, strategy, theme) => {
   const marks = computePieMarks(spec, scales, chartArea, strategy, false);
   const labels = computePieLabels(marks, chartArea, spec.labels.density, theme.colors.text);
-  for (let i = 0; i < marks.length && i < labels.length; i++) {
-    marks[i].label = labels[i];
+  for (const label of labels) {
+    if (label.index !== void 0 && marks[label.index]) marks[label.index].label = label;
   }
   return marks;
 };
 var donutRenderer = (spec, scales, chartArea, strategy, theme) => {
   const marks = computePieMarks(spec, scales, chartArea, strategy, true);
   const labels = computePieLabels(marks, chartArea, spec.labels.density, theme.colors.text);
-  for (let i = 0; i < marks.length && i < labels.length; i++) {
-    marks[i].label = labels[i];
+  for (const label of labels) {
+    if (label.index !== void 0 && marks[label.index]) marks[label.index].label = label;
   }
   return marks;
 };
@@ -13849,6 +13856,7 @@ var LEGEND_PADDING = 8;
 var LEGEND_RIGHT_WIDTH = 120;
 var RIGHT_LEGEND_MAX_HEIGHT_RATIO = 0.4;
 var TOP_LEGEND_MAX_ROWS = 2;
+var ARC_LABEL_CROWDING_LIMIT = 21;
 function hasLegendContent(content) {
   return content.entries.length > 0 || content.continuous != null;
 }
@@ -14006,6 +14014,12 @@ function computeLegendContent(spec, strategy, theme, availableWidth, availableHe
       }
     }
   }
+  const density = spec.labels.density;
+  const everySliceLabeled = density === "all" || density === "auto";
+  const uncrowded = seriesCount > 0 && seriesCount <= ARC_LABEL_CROWDING_LIMIT;
+  if (spec.markType === "arc" && everySliceLabeled && uncrowded && !spec.userExplicit?.legend) {
+    entries = [];
+  }
   const labelStyle = {
     fontFamily: theme.fonts.family,
     fontSize: theme.fonts.sizes.small,
@@ -16015,6 +16029,7 @@ function collectContrastWarnings(scales, markType, theme) {
 
 // src/compiler/mark-render-mode.ts
 var AUTO_CANVAS_THRESHOLD = 1e3;
+var AUTO_CANVAS_REFUSAL_WARN_THRESHOLD = 5e3;
 function resolveMarkRenderMode(args, warnings = []) {
   const { markType, pointCount, display, faceted, layered } = args;
   const requested = args.requested ?? "auto";
@@ -16025,6 +16040,10 @@ function resolveMarkRenderMode(args, warnings = []) {
       warnings.push(
         `Chart warning: renderer "canvas" is not supported ${refusal}; rendering marks as SVG instead.`
       );
+    } else if (pointCount > AUTO_CANVAS_REFUSAL_WARN_THRESHOLD) {
+      warnings.push(
+        `[openchart] Rendering ${pointCount} point marks as SVG: the canvas mark layer is not supported ${refusal}. Expect slow paint and interaction. Reduce the point count, or restructure so the dense marks compile as a single unlayered, unfaceted point chart.`
+      );
     }
     return "svg";
   }
@@ -24106,6 +24125,19 @@ function compileFaceted(chartSpec, renderSpec, facetChannel, facetDirection, opt
     chartSpec.data
   );
   assignAnimationIndices(allMarks, resolvedAnimation);
+  const facetRenderModeWarnings = [];
+  resolveMarkRenderMode(
+    {
+      requested: options.renderer,
+      markType: chartSpec.markType,
+      pointCount: allMarks.reduce((n, m) => m.type === "point" ? n + 1 : n, 0),
+      display: chartSpec.display,
+      faceted: true,
+      layered: options.layered ?? false
+    },
+    facetRenderModeWarnings
+  );
+  emitSpecWarnings(facetRenderModeWarnings, options.onWarn);
   return {
     area: chartArea,
     chrome: allFootnotes.length > 0 ? { ...dims.chrome, footnotes: allFootnotes } : dims.chrome,