fix(#8338): make boxplot work with single value per group

vega · Oct 5, 2023 · e1438dd · e1438dd
1 parent ff44fa8
commit e1438dd
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/src/compositemark/boxplot.ts b/src/compositemark/boxplot.ts
@@ -134,7 +134,17 @@ export function normalizeBoxPlot(
 
   const makeBoxPlotExtent = makeBoxPlotPart(encodingWithoutSizeColorAndContinuousAxis);
   const makeBoxPlotBox = makeBoxPlotPart(encodingWithoutContinuousAxis);
-  const makeBoxPlotMidTick = makeBoxPlotPart({...encodingWithoutSizeColorAndContinuousAxis, ...(size ? {size} : {})});
+  const defaultBoxColor = isObject(config.boxplot.box) ? config.boxplot.box.color : config.mark.color || '#4c78a8';
+  const makeBoxPlotMidTick = makeBoxPlotPart({
+    ...encodingWithoutSizeColorAndContinuousAxis,
+    ...(size ? {size} : {}),
+    color: {
+      condition: {
+        test: `datum['lower_box_${continuousAxisChannelDef.field}'] >= datum['upper_box_${continuousAxisChannelDef.field}']`,
+        ...(color || {value: defaultBoxColor})
+      }
+    }
+  });
 
   const fiveSummaryTooltipEncoding: Encoding<string> = getCompositeMarkTooltip(
     [