pmx-canvas 0.1.25 → 0.1.27

package/src/json-render/charts/components.tsx

@@ -68,6 +68,9 @@ export function processChartData(
   return result;
 }
 
+export type BarColorBy = 'series' | 'category' | 'value' | 'none';
+export type BarHighlight = number | 'max' | 'min' | null;
+
 export interface CartesianChartProps {
   title?: string | null;
   data: Record<string, unknown>[];
@@ -76,6 +79,10 @@ export interface CartesianChartProps {
   aggregate?: AggregateMode | null;
   color?: string | null;
   height?: number | null;
+  /** Bar-only: how bar fills are colored. Defaults to 'series'. Ignored by line charts. */
+  colorBy?: BarColorBy | null;
+  /** Bar-only: which bar gets the accent under colorBy='series'. Defaults to 'max'. */
+  highlight?: BarHighlight;
 }
 
 interface PieChartProps {
@@ -109,6 +116,7 @@ export const legendMargin = { top: 10 };
 export function useChartFrameHeight(explicitHeight: number | null | undefined, fallbackHeight = 300) {
   const frameRef = useRef<HTMLDivElement>(null);
   const [autoHeight, setAutoHeight] = useState(fallbackHeight);
+  const [autoWidth, setAutoWidth] = useState(0);
 
   useEffect(() => {
     const frame = frameRef.current;
@@ -121,6 +129,7 @@ export function useChartFrameHeight(explicitHeight: number | null | undefined, f
       const overflow = Math.max(0, doc.scrollHeight - doc.clientHeight);
       const available = overflow > 0 ? currentHeight - overflow : window.innerHeight - rect.top - 24;
       setAutoHeight(Math.max(220, Math.round(available)));
+      setAutoWidth(Math.round(rect.width));
     };
 
     updateHeight();
@@ -137,6 +146,7 @@ export function useChartFrameHeight(explicitHeight: number | null | undefined, f
   return {
     frameRef,
     height: typeof explicitHeight === 'number' ? Math.min(explicitHeight, autoHeight) : autoHeight,
+    width: autoWidth,
   };
 }
 
@@ -187,19 +197,96 @@ function ChartLineChart({ props }: BaseComponentProps<CartesianChartProps>) {
   );
 }
 
+/**
+ * Resolve the highlighted bar index for colorBy='series'.
+ * 'max'/'min' pick the tallest/shortest yKey value; a number is used as-is
+ * (clamped to range); null/out-of-range means no bar is emphasized.
+ */
+function resolveHighlightIndex(
+  data: Record<string, unknown>[],
+  yKey: string,
+  highlight: BarHighlight,
+): number {
+  if (highlight === null || data.length === 0) return -1;
+  if (typeof highlight === 'number') {
+    return highlight >= 0 && highlight < data.length ? highlight : -1;
+  }
+  let best = 0;
+  let bestVal = Number(data[0]?.[yKey] ?? 0);
+  for (let i = 1; i < data.length; i++) {
+    const val = Number(data[i]?.[yKey] ?? 0);
+    if (highlight === 'max' ? val > bestVal : val < bestVal) {
+      best = i;
+      bestVal = val;
+    }
+  }
+  return best;
+}
+
+/** Per-bar fill for each colorBy mode. Reuses the <Cell> pattern proven in ChartPieChart. */
+function barCellFill(
+  mode: BarColorBy,
+  accent: string,
+  index: number,
+  value: number,
+  range: { min: number; max: number },
+  highlightIndex: number,
+): string {
+  switch (mode) {
+    case 'category':
+      return CHART_COLORS[index % CHART_COLORS.length];
+    case 'value': {
+      // Sequential shade by magnitude: 35% (lowest) -> 100% (highest) accent,
+      // mixed toward a SOLID background token so every bar is opaque and the
+      // ramp is a true lightness sequence (not a translucency that reads
+      // differently depending on what is behind the bar).
+      const span = range.max - range.min;
+      const t = span > 0 ? (value - range.min) / span : 1;
+      const pct = Math.round(35 + t * 65);
+      return `color-mix(in oklch, ${accent} ${pct}%, var(--card, var(--background)))`;
+    }
+    case 'none':
+      return accent;
+    case 'series':
+    default:
+      // Tufte-safe emphasis: one accent bar, the rest a muted version of it.
+      return index === highlightIndex
+        ? accent
+        : `color-mix(in oklch, ${accent} 32%, transparent)`;
+  }
+}
+
 function ChartBarChart({ props }: BaseComponentProps<CartesianChartProps>) {
-  const fill = props.color ?? CHART_COLORS[0];
+  const accent = props.color ?? CHART_COLORS[0];
+  const mode: BarColorBy = props.colorBy ?? 'series';
+  const highlight: BarHighlight = props.highlight === undefined ? 'max' : props.highlight;
   return (
     <CartesianChart props={props} className="pmx-chart--bar">
-      {(data) => (
-        <RechartsBarChart data={data} margin={chartMargin}>
-          <CartesianGrid strokeDasharray="3 3" stroke="var(--border, #e5e5e5)" />
-          <XAxis dataKey={props.xKey} tick={axisStyle} tickMargin={axisTickMargin} />
-          <YAxis tick={axisStyle} tickMargin={axisTickMargin} />
-          <Tooltip contentStyle={tooltipStyle} cursor={false} />
-          <Bar dataKey={props.yKey} fill={fill} radius={[4, 4, 0, 0]} />
-        </RechartsBarChart>
-      )}
+      {(data) => {
+        const values = data.map((row) => Number(row[props.yKey] ?? 0));
+        const range = {
+          min: values.length ? Math.min(...values) : 0,
+          max: values.length ? Math.max(...values) : 0,
+        };
+        const highlightIndex =
+          mode === 'series' ? resolveHighlightIndex(data, props.yKey, highlight) : -1;
+        return (
+          <RechartsBarChart data={data} margin={chartMargin}>
+            <CartesianGrid strokeDasharray="3 3" stroke="var(--border, #e5e5e5)" />
+            <XAxis dataKey={props.xKey} tick={axisStyle} tickMargin={axisTickMargin} />
+            <YAxis domain={[0, 'auto']} tick={axisStyle} tickMargin={axisTickMargin} />
+            <Tooltip contentStyle={tooltipStyle} cursor={false} />
+            <Bar dataKey={props.yKey} radius={[4, 4, 0, 0]}>
+              {data.map((_, i) => (
+                <Cell
+                  key={i}
+                  fill={barCellFill(mode, accent, i, values[i], range, highlightIndex)}
+                />
+              ))}
+            </Bar>
+          </RechartsBarChart>
+        );
+      }}
     </CartesianChart>
   );
 }

package/src/json-render/charts/definitions.ts

@@ -17,6 +17,21 @@ const cartesianProps = z.object({
   height: z.number().nullable(),
 });
 
+const barChartProps = cartesianProps.extend({
+  colorBy: z
+    .enum(['series', 'category', 'value', 'none'])
+    .nullable()
+    .describe(
+      "How bars are colored. 'series' (default) = single accent with ONE highlighted bar (Tufte-safe emphasis); 'category' = rotate the categorical palette per bar (only when the x-axis category itself is the message); 'value' = sequential shade by magnitude; 'none' = flat single accent.",
+    ),
+  highlight: z
+    .union([z.number(), z.enum(['max', 'min'])])
+    .nullable()
+    .describe(
+      "For colorBy='series', which bar gets the accent: 'max' (default, tallest), 'min' (shortest), a 0-based index, or null for no emphasis. Ignored by other colorBy modes.",
+    ),
+});
+
 export const chartComponentDefinitions = {
   LineChart: {
     props: cartesianProps,
@@ -38,9 +53,9 @@ export const chartComponentDefinitions = {
   },
 
   BarChart: {
-    props: cartesianProps,
+    props: barChartProps,
     description:
-      'Bar chart for comparing categories. Provide data as an array of objects with xKey and yKey fields.',
+      "Bar chart for comparing categories. Provide data as an array of objects with xKey and yKey fields. Color encodes data, not decoration: by default one accent with the tallest bar highlighted (colorBy='series'). Set colorBy='category' only when the category itself is the message, 'value' to shade by magnitude, or 'none' for a flat fill.",
     example: {
       title: 'Sales by region',
       data: [
@@ -53,6 +68,8 @@ export const chartComponentDefinitions = {
       aggregate: null,
       color: null,
       height: null,
+      colorBy: 'series',
+      highlight: 'max',
     },
   },
 

package/src/json-render/charts/extra-components.tsx

@@ -55,20 +55,21 @@ function ChartAreaChart({ props }: BaseComponentProps<AreaChartProps>) {
         <RechartsAreaChart data={data} margin={chartMargin}>
           <defs>
             <linearGradient id={gradientId} x1="0" y1="0" x2="0" y2="1">
-              <stop offset="0%" stopColor={stroke} stopOpacity={0.45} />
-              <stop offset="100%" stopColor={stroke} stopOpacity={0.05} />
+              <stop offset="0%" stopColor={stroke} stopOpacity={0.55} />
+              <stop offset="100%" stopColor={stroke} stopOpacity={0.12} />
             </linearGradient>
           </defs>
           <CartesianGrid strokeDasharray="3 3" stroke="var(--border, #e5e5e5)" />
           <XAxis dataKey={props.xKey} tick={axisStyle} tickMargin={axisTickMargin} />
-          <YAxis tick={axisStyle} tickMargin={axisTickMargin} />
+          <YAxis domain={[0, 'auto']} tick={axisStyle} tickMargin={axisTickMargin} />
           <Tooltip contentStyle={tooltipStyle} />
           <Area
             type="monotone"
             dataKey={props.yKey}
             stroke={stroke}
-            strokeWidth={2}
+            strokeWidth={2.5}
             fill={`url(#${gradientId})`}
+            dot={{ r: 2.5, fill: stroke, strokeWidth: 0 }}
             activeDot={{ r: 5 }}
           />
         </RechartsAreaChart>

package/src/json-render/charts/tufte-components.tsx

@@ -0,0 +1,383 @@
+/** @jsxImportSource react */
+
+/**
+ * Tufte primitive chart components for json-render.
+ *
+ * Word-sized / high-data-ink-ratio primitives that Recharts handles poorly:
+ * Sparkline, DotPlot (Cleveland), BulletChart (Few), and Slopegraph (Tufte).
+ * Hand-rolled SVG keeps the data-ink ratio high and avoids Recharts chartjunk.
+ *
+ * Lives alongside ./components.tsx and ./extra-components.tsx so the original
+ * chart sets stay unchanged; ./catalog.ts is the only merge surface.
+ */
+
+import type { BaseComponentProps } from '@json-render/react';
+import { CHART_COLORS, useChartFrameHeight } from './components';
+
+const ACCENT = CHART_COLORS[0];
+const INK = 'var(--foreground, #111)';
+const MUTED = 'var(--muted-foreground, #666)';
+const FRAME = 'var(--border, #e5e5e5)';
+
+function toNumber(value: unknown): number {
+  const n = Number(value);
+  return Number.isFinite(n) ? n : 0;
+}
+
+function extentOf(values: number[]): { min: number; max: number } {
+  if (values.length === 0) return { min: 0, max: 1 };
+  let min = values[0];
+  let max = values[0];
+  for (const v of values) {
+    if (v < min) min = v;
+    if (v > max) max = v;
+  }
+  if (min === max) {
+    // Avoid a zero-height/width domain so the line/dot still renders.
+    return { min: min - 1, max: max + 1 };
+  }
+  return { min, max };
+}
+
+/* ───────────────────────── Sparkline ───────────────────────── */
+
+interface SparklineProps {
+  title?: string | null;
+  data: Record<string, unknown>[];
+  valueKey: string;
+  color?: string | null;
+  fill?: boolean | null;
+  showEndDot?: boolean | null;
+  showMinMax?: boolean | null;
+  showValue?: boolean | null;
+  height?: number | null;
+}
+
+function ChartSparkline({ props }: BaseComponentProps<SparklineProps>) {
+  const rows = props.data ?? [];
+  const values = rows.map((row) => toNumber(row[props.valueKey]));
+  const stroke = props.color ?? ACCENT;
+  const h = typeof props.height === 'number' ? props.height : 36;
+  const w = 240;
+  const padX = 4;
+  const padY = 4;
+
+  const { min, max } = extentOf(values);
+  const innerW = w - padX * 2;
+  const innerH = h - padY * 2;
+  const stepX = values.length > 1 ? innerW / (values.length - 1) : 0;
+  const scaleY = (v: number) => padY + innerH - ((v - min) / (max - min)) * innerH;
+  const points = values.map((v, i) => ({ x: padX + i * stepX, y: scaleY(v), v }));
+
+  const linePath = points.map((p, i) => `${i === 0 ? 'M' : 'L'}${p.x.toFixed(1)},${p.y.toFixed(1)}`).join(' ');
+  const areaPath =
+    points.length > 0
+      ? `${linePath} L${points[points.length - 1].x.toFixed(1)},${(padY + innerH).toFixed(1)} L${points[0].x.toFixed(1)},${(padY + innerH).toFixed(1)} Z`
+      : '';
+
+  let minIdx = 0;
+  let maxIdx = 0;
+  values.forEach((v, i) => {
+    if (v < values[minIdx]) minIdx = i;
+    if (v > values[maxIdx]) maxIdx = i;
+  });
+  const last = points[points.length - 1];
+  const lastValue = values.length > 0 ? values[values.length - 1] : 0;
+
+  return (
+    <div className="pmx-chart pmx-chart--sparkline">
+      {props.title && <div className="pmx-chart__title">{props.title}</div>}
+      <div className="pmx-chart__sparkline-row">
+        {/*
+          preserveAspectRatio="none" stretches the 240×36 viewBox to the cell.
+          Slope is faithful only when the cell's aspect ratio stays near 240:36;
+          this is acceptable for a word-sized strip whose job is shape, not exact angle.
+        */}
+        <svg
+          className="pmx-chart__sparkline-svg"
+          viewBox={`0 0 ${w} ${h}`}
+          preserveAspectRatio="none"
+          role="img"
+          aria-label={props.title ?? 'sparkline'}
+        >
+          {props.fill && areaPath && (
+            <path d={areaPath} fill={stroke} fillOpacity={0.12} stroke="none" />
+          )}
+          {points.length > 1 && (
+            <path d={linePath} fill="none" stroke={stroke} strokeWidth={1.5} vectorEffect="non-scaling-stroke" />
+          )}
+          {props.showMinMax && points.length > 0 && (
+            <>
+              <circle cx={points[minIdx].x} cy={points[minIdx].y} r={2} fill={MUTED} />
+              <circle cx={points[maxIdx].x} cy={points[maxIdx].y} r={2} fill={MUTED} />
+            </>
+          )}
+          {props.showEndDot !== false && last && <circle cx={last.x} cy={last.y} r={2.5} fill={stroke} />}
+        </svg>
+        {props.showValue && (
+          <span className="pmx-chart__sparkline-value" style={{ color: stroke }}>
+            {lastValue}
+          </span>
+        )}
+      </div>
+    </div>
+  );
+}
+
+/* ───────────────────────── DotPlot (Cleveland) ───────────────────────── */
+
+interface DotPlotProps {
+  title?: string | null;
+  data: Record<string, unknown>[];
+  labelKey: string;
+  valueKey: string;
+  color?: string | null;
+  sort?: 'asc' | 'desc' | 'none' | null;
+  height?: number | null;
+}
+
+function ChartDotPlot({ props }: BaseComponentProps<DotPlotProps>) {
+  const dot = props.color ?? ACCENT;
+  const rows = (props.data ?? []).map((row) => ({
+    label: String(row[props.labelKey] ?? ''),
+    value: toNumber(row[props.valueKey]),
+  }));
+  const sort = props.sort ?? 'desc';
+  if (sort === 'asc') rows.sort((a, b) => a.value - b.value);
+  else if (sort === 'desc') rows.sort((a, b) => b.value - a.value);
+
+  const fallback = Math.max(160, rows.length * 28 + 24);
+  const { frameRef, height } = useChartFrameHeight(props.height, fallback);
+
+  const values = rows.map((r) => r.value);
+  const { min, max } = extentOf(values);
+  const domainMin = Math.min(0, min);
+  const labelW = 140;
+  const valueW = 52;
+  const padX = 12;
+  const rowH = rows.length > 0 ? Math.min(36, (height - 12) / rows.length) : 24;
+  const plotLeft = labelW + padX;
+
+  return (
+    <div ref={frameRef} className="pmx-chart pmx-chart--dot-plot" style={{ height }}>
+      {props.title && <div className="pmx-chart__title">{props.title}</div>}
+      <svg className="pmx-chart__dot-plot-svg" width="100%" height={rows.length * rowH} role="img" aria-label={props.title ?? 'dot plot'}>
+        {rows.map((row, i) => {
+          const cy = i * rowH + rowH / 2;
+          // Reference rule runs from the axis origin to the dot, so the line's
+          // length itself encodes the value (a full-width rule would make every
+          // row look equal and fight the dot encoding).
+          const span = max - domainMin || 1;
+          const frac = Math.max(0, Math.min(1, (row.value - domainMin) / span));
+          const dotCx = `calc(${plotLeft}px + (100% - ${plotLeft + valueW + padX}px) * ${frac})`;
+          return (
+            <g key={`${row.label}-${i}`}>
+              <text x={labelW} y={cy} textAnchor="end" dominantBaseline="central" fontSize={12} fill={INK}>
+                {row.label}
+              </text>
+              <line x1={plotLeft} y1={cy} x2={dotCx} y2={cy} stroke={FRAME} strokeWidth={1} />
+              <circle cx={dotCx} cy={cy} r={4.5} fill={dot} />
+              <text x="100%" dx={-padX} y={cy} textAnchor="end" dominantBaseline="central" fontSize={12} fill={MUTED}>
+                {row.value}
+              </text>
+            </g>
+          );
+        })}
+      </svg>
+    </div>
+  );
+}
+
+/* ───────────────────────── BulletChart (Few) ───────────────────────── */
+
+interface BulletRow {
+  label?: string;
+  value: number;
+  target?: number;
+  ranges?: number[];
+}
+
+interface BulletChartProps {
+  title?: string | null;
+  data: Record<string, unknown>[];
+  labelKey?: string | null;
+  valueKey: string;
+  targetKey?: string | null;
+  rangesKey?: string | null;
+  color?: string | null;
+  height?: number | null;
+}
+
+function ChartBulletChart({ props }: BaseComponentProps<BulletChartProps>) {
+  const measure = props.color ?? ACCENT;
+  const labelKey = props.labelKey ?? 'label';
+  const targetKey = props.targetKey ?? 'target';
+  const rangesKey = props.rangesKey ?? 'ranges';
+
+  const rows: BulletRow[] = (props.data ?? []).map((row) => {
+    const rawRanges = row[rangesKey];
+    return {
+      label: String(row[labelKey] ?? ''),
+      value: toNumber(row[props.valueKey]),
+      ...(row[targetKey] !== undefined ? { target: toNumber(row[targetKey]) } : {}),
+      ...(Array.isArray(rawRanges) ? { ranges: rawRanges.map(toNumber).sort((a, b) => a - b) } : {}),
+    };
+  });
+
+  const fallback = Math.max(120, rows.length * 48 + 24);
+  const { frameRef, height, width } = useChartFrameHeight(props.height, fallback);
+  // SVG <text x> does not support calc(), so position everything in measured
+  // pixels (fallback width until the ResizeObserver reports the real size).
+  const w = width || 480;
+  const labelW = 120;
+  const padX = 12;
+  const rowH = rows.length > 0 ? Math.min(56, (height - 12) / rows.length) : 48;
+
+  return (
+    <div ref={frameRef} className="pmx-chart pmx-chart--bullet" style={{ height }}>
+      {props.title && <div className="pmx-chart__title">{props.title}</div>}
+      <svg className="pmx-chart__bullet-svg" width="100%" height={rows.length * rowH} role="img" aria-label={props.title ?? 'bullet chart'}>
+        {rows.map((row, i) => {
+          const top = i * rowH;
+          const cy = top + rowH / 2;
+          const domainMax =
+            Math.max(row.value, row.target ?? 0, ...(row.ranges ?? [0])) || 1;
+          const ranges = row.ranges ?? [];
+          const left = labelW + padX;
+          const rightInset = padX;
+          const plotW = Math.max(0, w - left - rightInset);
+          const pct = (v: number) => Math.max(0, Math.min(1, v / domainMax));
+          const xAt = (v: number) => left + plotW * pct(v);
+          const wBetween = (lo: number, hi: number) => plotW * Math.max(0, pct(hi) - pct(lo));
+          const barH = Math.min(20, rowH * 0.5);
+          const measureH = barH * 0.4;
+          // Qualitative bands: lightest (worst) to darkest (best) grayscale.
+          const bandShades = ['color-mix(in oklch, var(--muted) 35%, transparent)',
+                              'color-mix(in oklch, var(--muted) 60%, transparent)',
+                              'color-mix(in oklch, var(--muted) 90%, transparent)'];
+          return (
+            <g key={`${row.label}-${i}`}>
+              <text x={labelW} y={cy} textAnchor="end" dominantBaseline="central" fontSize={12} fill={INK}>
+                {row.label}
+              </text>
+              {ranges.map((hi, idx) => {
+                const lo = idx === 0 ? 0 : ranges[idx - 1];
+                return (
+                  <rect
+                    key={idx}
+                    x={xAt(lo)}
+                    y={cy - barH / 2}
+                    width={wBetween(lo, hi)}
+                    height={barH}
+                    fill={bandShades[Math.min(idx, bandShades.length - 1)]}
+                  />
+                );
+              })}
+              {/* Per-row scale ticks at each band boundary so the reader does not
+                  compare bar lengths across rows that may be independently scaled. */}
+              {ranges.map((hi, idx) => (
+                <text key={`tick-${idx}`} x={xAt(hi)} y={cy + barH / 2 + 10} textAnchor="middle" fontSize={9} fill={MUTED}>
+                  {hi}
+                </text>
+              ))}
+              {/* Measure bar (the actual value) — the only saturated ink. */}
+              <rect x={left} y={cy - measureH / 2} width={wBetween(0, row.value)} height={measureH} fill={measure} />
+              {/* Target tick. */}
+              {typeof row.target === 'number' && (
+                <rect x={xAt(row.target)} y={cy - barH / 2} width={2} height={barH} fill={INK} />
+              )}
+            </g>
+          );
+        })}
+      </svg>
+    </div>
+  );
+}
+
+/* ───────────────────────── Slopegraph ───────────────────────── */
+
+interface SlopegraphProps {
+  title?: string | null;
+  data: Record<string, unknown>[];
+  labelKey: string;
+  beforeKey: string;
+  afterKey: string;
+  beforeLabel?: string | null;
+  afterLabel?: string | null;
+  color?: string | null;
+  colorByDirection?: boolean | null;
+  height?: number | null;
+}
+
+function ChartSlopegraph({ props }: BaseComponentProps<SlopegraphProps>) {
+  const stroke = props.color ?? ACCENT;
+  const rows = (props.data ?? []).map((row) => ({
+    label: String(row[props.labelKey] ?? ''),
+    before: toNumber(row[props.beforeKey]),
+    after: toNumber(row[props.afterKey]),
+  }));
+
+  const { frameRef, height, width } = useChartFrameHeight(props.height, 320);
+  const all = rows.flatMap((r) => [r.before, r.after]);
+  const { min, max } = extentOf(all);
+  const topPad = 28;
+  const botPad = 20;
+  const leftX = 150;
+  const rightInset = 150;
+  // SVG <text x> does not support calc(), so position the right column in
+  // measured pixels (fallback width until the ResizeObserver reports the size).
+  const w = width || 480;
+  const rightX = Math.max(leftX + 40, w - rightInset);
+  const scaleY = (v: number) => topPad + (1 - (v - min) / (max - min)) * (height - topPad - botPad);
+
+  return (
+    <div ref={frameRef} className="pmx-chart pmx-chart--slopegraph" style={{ height }}>
+      {props.title && <div className="pmx-chart__title">{props.title}</div>}
+      <svg className="pmx-chart__slopegraph-svg" width="100%" height={height} role="img" aria-label={props.title ?? 'slopegraph'}>
+        <text x={leftX} y={14} textAnchor="end" fontSize={12} fontWeight={600} fill={MUTED}>
+          {props.beforeLabel ?? props.beforeKey}
+        </text>
+        <text x={rightX} y={14} textAnchor="start" fontSize={12} fontWeight={600} fill={MUTED}>
+          {props.afterLabel ?? props.afterKey}
+        </text>
+        {rows.map((row, i) => {
+          const y1 = scaleY(row.before);
+          const y2 = scaleY(row.after);
+          const rose = row.after >= row.before;
+          // Lines default to a single neutral ink. Direction coloring (rising vs
+          // falling) is an opt-in via colorByDirection — by default it would be a
+          // redundant double-encoding of slope and editorializes (a falling
+          // error-rate is "good", a falling revenue is "bad").
+          const lineColor = props.colorByDirection ? (rose ? stroke : MUTED) : stroke;
+          return (
+            <g key={`${row.label}-${i}`}>
+              <line
+                x1={leftX}
+                y1={y1}
+                x2={rightX}
+                y2={y2}
+                stroke={lineColor}
+                strokeWidth={1.5}
+              />
+              <circle cx={leftX} cy={y1} r={2.5} fill={lineColor} />
+              <circle cx={rightX} cy={y2} r={2.5} fill={lineColor} />
+              <text x={leftX - 8} y={y1} textAnchor="end" dominantBaseline="central" fontSize={11} fill={INK}>
+                {`${row.label}  ${row.before}`}
+              </text>
+              <text x={rightX + 8} y={y2} textAnchor="start" dominantBaseline="central" fontSize={11} fill={INK}>
+                {`${row.after}  ${row.label}`}
+              </text>
+            </g>
+          );
+        })}
+      </svg>
+    </div>
+  );
+}
+
+export const tufteChartComponents = {
+  Sparkline: ChartSparkline,
+  DotPlot: ChartDotPlot,
+  BulletChart: ChartBulletChart,
+  Slopegraph: ChartSlopegraph,
+};