ggaction 0.0.17 → 0.0.19

package/src/actions/guides/polar/resolve.js

@@ -1,9 +1,9 @@
 import { validateUserId } from "../../../core/identifiers.js";
 import {
   validateGeneratedItemLimit,
+  validateTextStyle,
   validateNonEmptyString,
-  validateNonNegativeFinite,
-  validatePositiveFinite
+  validateNonNegativeFinite
 } from "../../../core/validation.js";
 import {
   formatTransformedTick,
@@ -15,7 +15,7 @@ import { formatTimeTick, formatTimeTicks } from "../../../grammar/ticks.js";
 import { resolveCoordinatePolarFrame } from
   "../../../materialization/coordinateBounds.js";
 import { findCoordinate } from "../../../selectors/coordinates.js";
-import { DEFAULT_COLORS, DEFAULT_FONT_FAMILY } from
+import { AXIS_LABEL_FONT_SIZES, AXIS_TITLE_FONT_SIZE, DEFAULT_COLORS, DEFAULT_FONT_FAMILY } from
   "../../../theme/defaults.js";
 import {
   findGraphicParent,
@@ -41,7 +41,7 @@ export const POLAR_AXIS_DEFAULTS = Object.freeze({
     radiusOffset: 10,
     format: "auto",
     color: DEFAULT_COLORS.text,
-    fontSize: 11,
+    fontSize: AXIS_LABEL_FONT_SIZES.polar,
     fontFamily: DEFAULT_FONT_FAMILY,
     fontWeight: "normal"
   }),
@@ -49,7 +49,7 @@ export const POLAR_AXIS_DEFAULTS = Object.freeze({
     thetaOffset: 42,
     radiusOffset: 8,
     color: DEFAULT_COLORS.strongText,
-    fontSize: 13,
+    fontSize: AXIS_TITLE_FONT_SIZE,
     fontFamily: DEFAULT_FONT_FAMILY,
     fontWeight: 600
   })
@@ -264,15 +264,7 @@ export function validatePolarLineStyle(config, label) {
   validateNonNegativeFinite(config.lineWidth, `${label} lineWidth`);
 }
 
-export function validatePolarTextStyle(config, label) {
-  validateNonEmptyString(config.color, `${label} color`);
-  validatePositiveFinite(config.fontSize, `${label} fontSize`);
-  validateNonEmptyString(config.fontFamily, `${label} fontFamily`);
-  if (typeof config.fontWeight !== "string" &&
-      !Number.isFinite(config.fontWeight)) {
-    throw new TypeError(`${label} fontWeight must be a string or number.`);
-  }
-}
+export { validateTextStyle as validatePolarTextStyle };
 
 export function validatePolarLabelFormat(format) {
   return validateAxisFormat(format);

package/src/actions/guides/tickValues.js

@@ -1,3 +1,4 @@
+import { cartesianAxisOwners } from "../../selectors/axes.js";
 import { resolveHistogramBins } from "../../grammar/histogram.js";
 import {
   isTransformedScaleType,
@@ -61,12 +62,14 @@ export function valuesFromTickConfig(program, config) {
 }
 
 export function inferGridTickConfig(program, channel, scaleId) {
-  const axis = program.guideConfigs.axis?.[channel]?.ticks;
-  if (axis?.scale === scaleId) {
-    return axis.mode === "values"
-      ? { mode: "values", values: axis.values }
-      : { mode: "count", count: axis.count };
+  const axes = cartesianAxisOwners(program).filter(owner => owner.channel === channel &&
+    owner.config?.ticks?.scale === scaleId).map(owner => owner.config.ticks);
+  const policies = axes.map(axis => axis.mode === "values"
+    ? { mode: "values", values: axis.values } : { mode: "count", count: axis.count });
+  if (new Set(policies.map(policy => JSON.stringify(policy))).size > 1) {
+    throw new Error("Grid values require explicit count or values when bound axis tick policies differ.");
   }
+  if (policies.length > 0) return policies[0];
 
   const boundaries = inferHistogramBoundaries(program, channel, scaleId);
   const scale = program.resolvedScales[scaleId];

package/src/actions/index.js

@@ -1,27 +1,27 @@
-import { registerTextMetricActions } from "./textMetrics/index.js";
 import { registerPrimitiveActions } from "./primitives/index.js";
-import { registerSelectionActions } from "./selection/index.js";
-import { registerDataActions } from "./data/index.js";
-import { registerCategoryOrderActions } from "./categoryOrder/index.js";
-import { registerEncodingActions } from "./encodings/index.js";
-import { registerScaleActions } from "./scales/index.js";
-import { registerMarkActions } from "./marks/index.js";
+import { registerCanvasActions } from "./canvas/index.js";
 import { registerBasicCoordinateActions } from "./coordinates/actions.js";
 import { registerCoordinateEditActions } from "./coordinates/edit.js";
-import { registerFacetActions } from "./facets/index.js";
-import { registerCompositionActions } from "./composition/index.js";
-import { registerTitleActions } from "./titles/index.js";
-import { registerCanvasActions } from "./canvas/index.js";
+import { registerDataActions } from "./data/index.js";
+import { registerMarkActions } from "./marks/index.js";
+import { registerScaleActions } from "./scales/index.js";
+import { registerEncodingActions } from "./encodings/index.js";
 import { registerGuideActions } from "./guides/index.js";
-import { registerGradientPlotActions } from "./gradientPlots/index.js";
-import { registerBoxPlotActions } from "./boxPlots/index.js";
-import { registerErrorBarActions } from "./errorBars/index.js";
+import { registerTitleActions } from "./titles/index.js";
 import { registerRegressionActions } from "./regression/index.js";
-import { registerViolinPlotActions } from "./violinPlots/index.js";
+import { registerErrorBarActions } from "./errorBars/index.js";
 import { registerErrorBandActions } from "./errorBands/index.js";
+import { registerBoxPlotActions } from "./boxPlots/index.js";
+import { registerGradientPlotActions } from "./gradientPlots/index.js";
+import { registerViolinPlotActions } from "./violinPlots/index.js";
 import { registerChartActions } from "./charts/index.js";
-import { registerThemeActions } from "./theme/index.js";
+import { registerCategoryOrderActions } from "./categoryOrder/index.js";
+import { registerSelectionActions } from "./selection/index.js";
 import { registerResourceActions } from "./resources/index.js";
+import { registerTextMetricActions } from "./textMetrics/index.js";
+import { registerCompositionActions } from "./composition/index.js";
+import { registerFacetActions } from "./facets/index.js";
+import { registerThemeActions } from "./theme/index.js";
 import {
   applyCompositionTheme,
   removeCompositionTheme

package/src/actions/marks/arc/actions.js

@@ -1,3 +1,4 @@
+import { normalizeArcInnerRadius } from "../../../grammar/arcs.js";
 import { action } from "../../../core/action.js";
 import { validateUserId } from "../../../core/identifiers.js";
 import {
@@ -49,14 +50,6 @@ const CREATE_OPTIONS = Object.freeze(["id", "data", ...ARC_OPTIONS]);
 const EDIT_OPTIONS = Object.freeze(["target", ...ARC_OPTIONS]);
 const REMATERIALIZE_OPTIONS = Object.freeze(["id", "scales"]);
 
-function validateInnerRadius(value) {
-  if (!Number.isFinite(value) || value < 0 || value >= 1) {
-    throw new RangeError(
-      "Arc innerRadius must be from 0 (inclusive) to 1 (exclusive)."
-    );
-  }
-  return value;
-}
 
 function normalizeConfig(
   args,
@@ -67,7 +60,7 @@ function normalizeConfig(
     ...previous,
     ...strokeDetails,
     ...(Object.hasOwn(args, "innerRadius")
-      ? { innerRadius: validateInnerRadius(args.innerRadius), innerRadiusExplicit: true }
+      ? { innerRadius: normalizeArcInnerRadius(args.innerRadius), innerRadiusExplicit: true }
       : {}),
     ...(Object.hasOwn(args, "padAngle")
       ? { padAngle: validateNonNegativeFinite(args.padAngle, "Arc padAngle") }
@@ -204,7 +197,7 @@ const rematerializeArcMark = /* @__PURE__ */ action(
         ? {}
         : { radiusScale: resolved.resolvedScales[radiusScaleId] }),
       frame,
-      innerRadiusRatio: config.innerRadius ?? 0
+      innerRadius: config.innerRadius ?? 0
     });
     const commands = derived.sectors.map(sector => buildAnnularSectorCommands({
       frame,

package/src/actions/marks/area/actions.js

@@ -1,3 +1,5 @@
+import { validateFillPaint } from "../../../grammar/paint.js";
+import { validateAreaPaintConfig } from "../../../grammar/areaPaint.js";
 import { validateAreaMissing } from "../../../grammar/areaEndpoints.js";
 import { action } from "../../../core/action.js";
 import { validateUserId } from "../../../core/identifiers.js";
@@ -81,7 +83,7 @@ const createAreaMark = /* @__PURE__ */ action(
     if (missing === "break" && ["density", "horizon"].some(type => findUpstreamTransform(this, findDataset(this, data), type))) {
       throw new Error("Area missing:break does not reinterpret density or horizon data.");
     }
-    const fill = validateNonEmptyString(args.fill ?? DEFAULT_COLORS.mark, "Area fill");
+    const fill = validateFillPaint(args.fill ?? DEFAULT_COLORS.mark, "Area fill");
     const opacity = validateUnitInterval(args.opacity ?? 0.2, "Area opacity");
     const curve = validateCurveInterpolation(args.curve ?? "linear");
     const { stroke, strokeWidth } = validateAreaCreateOutline(args);
@@ -127,14 +129,14 @@ const configureAreaStrokeFromFill = /* @__PURE__ */ action(
     }
     const { stroke: _stroke, ...config } = this.markConfigs[id] ?? {};
     void _stroke;
-    return this._withMarkConfig(id, {
+    return this._withMarkConfig(id, validateAreaPaintConfig({
       ...config,
       strokeFromFill: true,
       strokeWidth: validateNonNegativeFinite(
         args.strokeWidth ?? 1,
         "Area strokeWidth"
       )
-    });
+    }));
   }
 );
 
@@ -199,7 +201,7 @@ const rematerializeAreaMark = /* @__PURE__ */ action(
     if (args.scales !== false && strokeEncoding?.scale !== undefined) {
       resolved = resolved.rematerializeScale({ id: strokeEncoding.scale });
     }
-    const config = this.markConfigs[id];
+    const config = validateAreaPaintConfig(this.markConfigs[id]);
     const { paths, fills, strokes } = resolveAreaMaterialization({
       rows: dataset.values,
       layer,
@@ -273,7 +275,7 @@ const editAreaMark = /* @__PURE__ */ action(
     let config = { ...this.markConfigs[layer.id] };
     Object.assign(config, strokeDetails);
     if (Object.hasOwn(args, "fill")) {
-      config.fill = validateNonEmptyString(args.fill, "Area fill");
+      config.fill = validateFillPaint(args.fill, "Area fill");
       if (config.errorBand !== undefined) config.errorBand = { ...config.errorBand, fill: config.fill };
     }
     if (Object.hasOwn(args, "opacity")) {
@@ -314,6 +316,7 @@ const editAreaMark = /* @__PURE__ */ action(
       );
     }
 
+    validateAreaPaintConfig(config);
     let next = this._withMarkConfig(layer.id, config);
     if (Object.hasOwn(args, "missing")) {
       const missing = validateAreaMissing(args.missing);

package/src/actions/marks/area/materialize.js

@@ -6,6 +6,7 @@ import {
 } from "../../../grammar/areaSeries.js";
 import {
   mapContinuousScaleValues,
+  mapOrdinalPositionValues,
   mapOrdinalValues
 } from "../../../grammar/scales/index.js";
 import { mapScaleConsumerValues } from
@@ -37,9 +38,8 @@ function resolveAreaPaths({
   }
   return derived.series.map(series => {
     if (densityTransform === undefined && derived.orientation === "horizontal") {
-      const y = mapContinuousScaleValues(
-        series.values.map(value => value.y),
-        yScale
+      const y = (["point", "band"].includes(yScale.type) ? mapOrdinalPositionValues : mapContinuousScaleValues)(
+        series.values.map(value => value.y), yScale
       );
       const lower = mapContinuousScaleValues(
         series.values.map(value => value.x),
@@ -57,9 +57,8 @@ function resolveAreaPaths({
       );
     }
     if (densityTransform !== undefined && derived.mode === "x-density") {
-      const y = mapContinuousScaleValues(
-        series.values.map(value => value.y),
-        yScale
+      const y = (["point", "band"].includes(yScale.type) ? mapOrdinalPositionValues : mapContinuousScaleValues)(
+        series.values.map(value => value.y), yScale
       );
       const upper = mapContinuousScaleValues(
         series.values.map(value => value.x),
@@ -80,7 +79,7 @@ function resolveAreaPaths({
         { independentAxis: "y" }
       );
     }
-    const x = mapContinuousScaleValues(
+    const x = (["point", "band"].includes(xScale.type) ? mapOrdinalPositionValues : mapContinuousScaleValues)(
       series.values.map(value => value.x),
       xScale
     );
@@ -134,8 +133,8 @@ export function resolveAreaMaterialization({
     layer.encoding?.y?.stack === "center";
   const rawDerived = densityTransform === undefined
     ? centered
-      ? deriveCenteredAreaSeries(rows, layer)
-      : deriveAreaSeries(rows, layer)
+      ? deriveCenteredAreaSeries(rows, layer, { xDomain: xScale.domain })
+      : deriveAreaSeries(rows, layer, { xDomain: xScale.domain, yDomain: yScale.domain })
     : deriveDensityAreaSeries(rows, layer, densityTransform);
   const colorEncoding = layer.encoding?.color;
   const layout = layer.layout?.mode ?? colorEncoding?.layout ?? "overlay";

package/src/actions/marks/bar/create.js

@@ -10,15 +10,15 @@ import {
 } from "../shared.js";
 import { resolveMarkGraphicPlacement } from
   "../../../materialization/graphicHierarchy.js";
-import { requestedRectStyleDetails } from
+import { requestedRectStyleDetails, RECT_RADIUS_PROPERTIES } from
   "../../../grammar/roundedRect.js";
 import { STROKE_STYLE_PROPERTIES } from
   "../../../grammar/strokeStyle.js";
 import { validateItemMissing } from "../../../grammar/itemMissing.js";
 
 const CREATE_OPTIONS = Object.freeze([
-  "id", "data", "missing", "fill", "opacity", "stroke", "strokeWidth",
-  "cornerRadius", ...STROKE_STYLE_PROPERTIES
+  "id", "data", "orientation", "missing", "fill", "opacity", "stroke", "strokeWidth",
+  ...RECT_RADIUS_PROPERTIES, ...STROKE_STYLE_PROPERTIES
 ]);
 
 export const createBarMark = /* @__PURE__ */ action(
@@ -29,6 +29,9 @@ export const createBarMark = /* @__PURE__ */ action(
   function (args = {}) {
     validateMarkOptions(args, CREATE_OPTIONS, "createBarMark");
     requestedRectStyleDetails(args, "createBarMark");
+    if (args.orientation !== undefined && !["vertical", "horizontal"].includes(args.orientation)) {
+      throw new Error("Bar orientation must be vertical or horizontal.");
+    }
     const id = resolveMarkId(this, args.id, {
       defaultId: "bar",
       label: "Bar mark id",
@@ -59,6 +62,9 @@ export const createBarMark = /* @__PURE__ */ action(
         value: validateItemMissing(args.missing, "Bar missing")
       });
     }
+    if (args.orientation !== undefined) {
+      created = created.editSemantic({ property: `layer[${id}].mark.orientation`, value: args.orientation });
+    }
     created = applyLayeredMarkInheritance(created, id, inherited);
     created = created
       .createGraphics({
@@ -70,7 +76,7 @@ export const createBarMark = /* @__PURE__ */ action(
     created = materializeInheritedMark(created, id);
     const appearance = Object.fromEntries(
       [
-        "fill", "opacity", "stroke", "strokeWidth", "cornerRadius",
+        "fill", "opacity", "stroke", "strokeWidth", ...RECT_RADIUS_PROPERTIES,
         ...STROKE_STYLE_PROPERTIES
       ]
         .filter(property => Object.hasOwn(args, property))

package/src/actions/marks/bar/edit.js

@@ -9,28 +9,29 @@ import { canMaterializeBar } from "../../../materialization/marks/index.js";
 import { DEFAULT_BAR_STROKE_WIDTH } from "../../../materialization/bars/resolve.js";
 import { resolveEligibleLayer } from "../../../selectors/layers.js";
 import { validateMarkOptions } from "../shared.js";
-import { requestedRectStyleDetails } from
+import { requestedRectStyleDetails, RECT_RADIUS_PROPERTIES } from
   "../../../grammar/roundedRect.js";
 import { STROKE_STYLE_PROPERTIES } from
   "../../../grammar/strokeStyle.js";
 import { rematerializeExistingLegend } from "../../encodings/shared.js";
+import { validateFillPaint } from "../../../grammar/paint.js";
 import { validateItemMissing } from "../../../grammar/itemMissing.js";
 
 const EDIT_OPTIONS = Object.freeze([
-  "target", "missing", "fill", "opacity", "stroke", "strokeWidth", "cornerRadius",
+  "target", "missing", "blend", "fill", "opacity", "stroke", "strokeWidth", ...RECT_RADIUS_PROPERTIES,
   ...STROKE_STYLE_PROPERTIES
 ]);
 
 export const editBarMark = /* @__PURE__ */ action(
   {
     op: "editBarMark",
-    description: "Edit whole-bar fill, opacity, and outline appearance."
+    description: "Edit whole-bar fill, opacity, outline, and compositing appearance."
   },
   function (args = {}) {
     validateMarkOptions(args, EDIT_OPTIONS, "editBarMark");
     const styleDetails = requestedRectStyleDetails(args, "editBarMark");
     const changes = [
-      "missing", "fill", "opacity", "stroke", "strokeWidth", "cornerRadius",
+      "missing", "blend", "fill", "opacity", "stroke", "strokeWidth", ...RECT_RADIUS_PROPERTIES,
       ...STROKE_STYLE_PROPERTIES
     ];
     if (!changes.some(key => Object.hasOwn(args, key))) {
@@ -56,23 +57,25 @@ export const editBarMark = /* @__PURE__ */ action(
         "editBarMark stroke conflicts with a field encoding; use encodeStroke with value to replace it."
       );
     }
+    if (Object.hasOwn(args, "opacity") && layer.encoding?.opacity !== undefined) {
+      throw new Error("editBarMark opacity conflicts with a field encoding; use encodeOpacity with value to replace it.");
+    }
     if (args.stroke === false && Object.hasOwn(args, "strokeWidth")) {
       throw new Error(
         "editBarMark cannot set strokeWidth while removing stroke."
       );
     }
 
-    const semantic = Object.hasOwn(args, "missing")
-      ? this.editSemantic({
-          property: `layer[${layer.id}].mark.missing`,
-          value: validateItemMissing(args.missing, "Bar missing")
-        })
-      : this;
-    const config = { ...semantic.markConfigs[layer.id] };
+    if (Object.hasOwn(args, "blend") && !["normal", "difference"].includes(args.blend)) {
+      throw new Error("Bar blend must be normal or difference.");
+    }
+    const missing = Object.hasOwn(args, "missing")
+      ? validateItemMissing(args.missing, "Bar missing") : undefined;
+    const config = { ...this.markConfigs[layer.id] };
     const appearance = { ...config.barAppearance };
     Object.assign(appearance, styleDetails);
     if (Object.hasOwn(args, "fill")) {
-      appearance.fill = validateNonEmptyString(args.fill, "Bar fill");
+      appearance.fill = validateFillPaint(args.fill, "Bar fill");
     }
     if (Object.hasOwn(args, "opacity")) {
       appearance.opacity = validateUnitInterval(args.opacity, "Bar opacity");
@@ -102,10 +105,22 @@ export const editBarMark = /* @__PURE__ */ action(
       );
     }
 
-    const next = semantic._withMarkConfig(layer.id, {
+    let semantic = missing === undefined ? this : this.editSemantic({
+      property: `layer[${layer.id}].mark.missing`, value: missing
+    });
+    if (Object.hasOwn(args, "blend")) {
+      if (args.blend === "difference") {
+        semantic = semantic.editSemantic({property: `layer[${layer.id}].mark.blend`, value: "difference"});
+      } else if (layer.mark.blend !== undefined) {
+        semantic = semantic.editSemantic({property: `layer[${layer.id}].mark.blend`, remove: true});
+      }
+    }
+    const editsAppearance = changes.some(key => key !== "missing" && key !== "blend" &&
+      Object.hasOwn(args, key));
+    const next = editsAppearance ? semantic._withMarkConfig(layer.id, {
       ...config,
       barAppearance: appearance
-    });
+    }) : semantic;
     const materialized = canMaterializeBar(next, layer)
       ? next.rematerializeBarMark({ id: layer.id })
       : next;

package/src/actions/marks/bar/materialize.js

@@ -1,3 +1,4 @@
+import { findLayer } from "../../../selectors/layers.js";
 import { editGraphicProperties } from "../../primitives/graphicProperties.js";
 import { action } from "../../../core/action.js";
 import { validateUserId } from "../../../core/identifiers.js";
@@ -13,19 +14,21 @@ import {
 import { rematerializeHighlightBaseline } from "../lifecycle.js";
 import { offsetCategoryRectangles } from
   "../../../materialization/categorySlotOffset.js";
-import { materializeRectItem } from "../../../grammar/roundedRect.js";
-import { STROKE_STYLE_PROPERTIES } from
-  "../../../grammar/strokeStyle.js";
+import { materializeRectItem, RECT_STYLE_PROPERTIES } from "../../../grammar/roundedRect.js";
 import { replaceGraphicItems } from "../../primitives/editGraphics.js";
 
 const REMATERIALIZE_OPTIONS = Object.freeze(["id", "scales"]);
 
 function editRectangles(program, id, rectangles) {
   const appearance = program.markConfigs[id]?.barAppearance ?? {};
-  const hasShapeStyle = ["cornerRadius", ...STROKE_STYLE_PROPERTIES]
+  const blend = findLayer(program, id)?.mark?.blend;
+  if (blend !== undefined) rectangles = rectangles.map(rect => ({...rect, blend}));
+  const hasShapeStyle = RECT_STYLE_PROPERTIES
     .some(property => Object.hasOwn(appearance, property));
-  if (hasShapeStyle) {
-    const radius = appearance.cornerRadius ?? 0;
+  const previousBlend = program.graphicSpec.objects[id]?.items?.some(item =>
+    item.properties?.blend !== undefined);
+  if (hasShapeStyle || blend !== undefined || previousBlend) {
+    const radius = appearance;
     return replaceGraphicItems(
       program,
       id,
@@ -92,6 +95,11 @@ export const rematerializeBarMark = /* @__PURE__ */ action(
       resolved = resolved.rematerializeScale({ id: strokeScaleId });
     }
 
+    const opacityScaleId = required.layer.encoding?.opacity?.scale;
+    if (opacityScaleId !== undefined && args.scales !== false) {
+      resolved = resolved.rematerializeScale({ id: opacityScaleId });
+    }
+
     if (required.materialization === "aggregate") {
       const offsetChannel = resolveBarOffsetChannel(required.layer);
       const offsetScaleId = required.layer.encoding?.[offsetChannel]?.scale;
@@ -104,7 +112,7 @@ export const rematerializeBarMark = /* @__PURE__ */ action(
       ));
     }
 
-    if (required.materialization === BAR_GRAINS.ranged) {
+    if ([BAR_GRAINS.ranged, BAR_GRAINS.centered].includes(required.materialization)) {
       const width = resolved.markConfigs[id]?.barWidth;
       return editRectangles(resolved, id, offsetCategoryRectangles(
         resolved, required.layer, deriveRangedRectangles(required, resolved, width)

package/src/actions/marks/line/actions.js

@@ -18,7 +18,7 @@ import {
 import { DEFAULT_COLORS } from "../../../theme/defaults.js";
 import { findDataset } from "../../../selectors/datasets.js";
 import { findLayer, resolveEligibleLayer } from "../../../selectors/layers.js";
-import { validateCurveInterpolation } from "../../../grammar/curveCommands.js";
+import { validateCurveInterpolation, validateCardinalTension } from "../../../grammar/curveCommands.js";
 import {
   resolveCoordinateBounds,
   resolveCoordinatePolarFrame
@@ -43,11 +43,11 @@ import {
 const DEFAULT_LINE_STROKE = DEFAULT_COLORS.mark;
 const DEFAULT_LINE_WIDTH = 2;
 const CREATE_OPTIONS = Object.freeze([
-  "id", "data", "stroke", "strokeWidth", "opacity", "curve", "closed",
+  "id", "data", "stroke", "strokeWidth", "opacity", "curve", "tension", "closed",
   ...STROKE_STYLE_PROPERTIES
 ]);
 const EDIT_OPTIONS = Object.freeze([
-  "target", "stroke", "strokeWidth", "opacity", "curve", "closed",
+  "target", "stroke", "strokeWidth", "opacity", "curve", "tension", "closed",
   ...STROKE_STYLE_PROPERTIES
 ]);
 const REMATERIALIZE_OPTIONS = Object.freeze(["id", "scales"]);
@@ -118,6 +118,7 @@ const createLineMark = /* @__PURE__ */ action(
       "Line strokeWidth"
     );
     const curve = validateCurveInterpolation(args.curve ?? "linear");
+    validateCardinalTension(args.tension, curve);
     const closed = Object.hasOwn(args, "closed")
       ? validateClosed(args.closed)
       : false;
@@ -151,6 +152,7 @@ const createLineMark = /* @__PURE__ */ action(
         {
           ...(Object.hasOwn(args, "strokeWidth") ? { strokeWidth } : {}),
           ...(Object.hasOwn(args, "curve") ? { curve } : {}),
+          ...(args.tension === undefined ? {} : { tension: args.tension }),
           ...(Object.hasOwn(args, "closed") ? { closed } : {}),
           ...strokeDetails
         }
@@ -277,7 +279,7 @@ const editLineMark = /* @__PURE__ */ action(
     validateMarkOptions(args, EDIT_OPTIONS, "editLineMark");
     const strokeDetails = requestedStrokeDetails(args, "editLineMark");
     if (!EDIT_OPTIONS.slice(1).some(key => Object.hasOwn(args, key))) {
-      throw new Error("editLineMark requires stroke, strokeWidth, opacity, curve, or closed.");
+      throw new Error("editLineMark requires stroke, strokeWidth, opacity, curve, tension, or closed.");
     }
     const target = Object.hasOwn(args, "target")
       ? validateUserId(args.target, "Line mark id")
@@ -315,6 +317,11 @@ const editLineMark = /* @__PURE__ */ action(
     })) {
       if (Object.hasOwn(args, key)) config[key] = validate(args[key], `Line ${key}`);
     }
+    if (args.curve !== undefined && args.curve !== "cardinal" && args.tension === undefined) {
+      delete config.tension;
+    }
+    if (Object.hasOwn(args, "tension")) config.tension = args.tension;
+    validateCardinalTension(config.tension, config.curve ?? "linear");
     validatePolarLineConfig(layer, config);
     const next = this._withMarkConfig(layer.id, config);
     const materialized = canMaterializeLine(next, layer)

package/src/actions/marks/line/materialize.js

@@ -3,6 +3,7 @@ import {
 } from "../../../grammar/lineSeries.js";
 import {
   mapContinuousScaleValues,
+  mapOrdinalPositionValues,
   normalizeStrokeDashPattern
 } from "../../../grammar/scales/index.js";
 import { buildCurvePathCommands } from
@@ -97,9 +98,7 @@ export function resolvePositionedLineMaterialization({
   const radiusScaleId = layer.encoding?.radius?.scale;
   const lineOptions = polar
     ? { thetaDomain: resolvedScales[thetaScaleId].domain }
-    : layer.encoding?.x?.bin === undefined
-      ? undefined
-      : { xBinBoundaries };
+    : { xBinBoundaries, xDomain: resolvedScales[xScaleId].domain };
   const derived = deriveLineSeries(rows, layer, lineOptions);
   const commands = polar
     ? derived.series.map(series => buildPolarLinePathCommands({
@@ -111,7 +110,9 @@ export function resolvePositionedLineMaterialization({
         closed: config.closed ?? false
       }))
     : derived.series.map(series => {
-        const x = mapContinuousScaleValues(
+        const mapX = ["nominal", "ordinal"].includes(layer.encoding.x.fieldType)
+          ? mapOrdinalPositionValues : mapContinuousScaleValues;
+        const x = mapX(
           series.values.map(value => value.x),
           resolvedScales[xScaleId]
         );
@@ -119,9 +120,12 @@ export function resolvePositionedLineMaterialization({
           series.values.map(value => value.y),
           resolvedScales[yScaleId]
         );
+        // A raw singleton retains its data position and series identity, but
+        // contributes no segment. Do not fabricate another observation.
+        if (series.values.length === 1) return [{ op: "M", x: x[0], y: y[0] }];
         return buildCurvePathCommands(
           series.values.map((_, index) => ({ x: x[index], y: y[index] })),
-          config.curve ?? "linear"
+          config.curve ?? "linear", config.tension
         );
       });
   const appearance = appearanceMapper(layer, resolvedScales,

package/src/actions/marks/point/edit.js

@@ -96,7 +96,7 @@ export const editPointMark = /* @__PURE__ */ action(
       }
     }
     if (Object.hasOwn(args, "strokeWidth")) {
-      if (typeof config.stroke !== "string") {
+      if (typeof config.stroke !== "string" && layer.encoding?.stroke?.field === undefined) {
         throw new Error("Point strokeWidth requires an active stroke.");
       }
       config.strokeWidth = validateNonNegativeFinite(

package/src/actions/marks/point/materialize.js

@@ -1,3 +1,4 @@
+import { validateGraphicItemLimit } from "../../../core/validation.js";
 import { editGraphicProperties } from "../../primitives/graphicProperties.js";
 import { action } from "../../../core/action.js";
 import { validateUserId } from "../../../core/identifiers.js";
@@ -29,6 +30,8 @@ import {
 } from "../../../theme/defaults.js";
 import { findDataset } from "../../../selectors/datasets.js";
 import { findLayer } from "../../../selectors/layers.js";
+import { resolveGraphicItemCapacity } from "../../../selectors/graphicCapacity.js";
+import { getMarkGraphicTypes } from "../../../core/vocabulary.js";
 import { canMaterializePoint } from "../../../materialization/marks/capabilities.js";
 import { rematerializeHighlightBaseline } from "../lifecycle.js";
 import {
@@ -335,13 +338,14 @@ export const rematerializePointMark = /* @__PURE__ */ action(
     if (layer?.mark?.type !== "point") {
       throw new Error(`Unknown point mark "${id}".`);
     }
-    if (!["circle", "rect", "path", "collection"].includes(graphic?.type)) {
+    if (!getMarkGraphicTypes("point").includes(graphic?.type)) {
       throw new Error(`Point mark "${id}" requires point graphics.`);
     }
     const sourceDataset = findDataset(resolved, layer.data);
     if (sourceDataset === undefined) {
       throw new Error(`Point mark "${id}" requires an existing dataset.`);
     }
+    validateGraphicItemLimit(sourceDataset.values.length, resolveGraphicItemCapacity(resolved, { id, label: "Point mark item count" }));
     const dataset = applyItemMissingPolicy(layer, sourceDataset);
 
     if (graphic.items.length === 0 && !canMaterializePoint(resolved, layer)) {

package/src/actions/marks/rect/actions.js

@@ -25,7 +25,7 @@ import { resolveMarkGraphicPlacement } from
 import { rematerializeExistingLegend } from "../../encodings/shared.js";
 import { rematerializeHighlightBaseline } from "../lifecycle.js";
 import {
-  requestedRectStyleDetails
+  requestedRectStyleDetails, hasRectRadius, RECT_RADIUS_PROPERTIES
 } from "../../../grammar/roundedRect.js";
 import { STROKE_STYLE_PROPERTIES } from
   "../../../grammar/strokeStyle.js";
@@ -33,7 +33,7 @@ import { applyItemMissingPolicy, validateItemMissing } from
   "../../../grammar/itemMissing.js";
 
 const STYLE_OPTIONS = Object.freeze([
-  "fill", "opacity", "stroke", "strokeWidth", "cornerRadius",
+  "fill", "opacity", "stroke", "strokeWidth", ...RECT_RADIUS_PROPERTIES,
   ...STROKE_STYLE_PROPERTIES
 ]);
 const CREATE_OPTIONS = Object.freeze(["id", "data", "missing", ...STYLE_OPTIONS]);
@@ -117,7 +117,7 @@ const rematerializeRectMark = /* @__PURE__ */ action(
     const graphic = this.graphicSpec.objects[id];
     if (layer?.mark?.type !== "rect") throw new Error(`Unknown rect mark "${id}".`);
     const roundedCollection = graphic?.type === "collection" &&
-      Object.hasOwn(this.markConfigs[id] ?? {}, "cornerRadius") &&
+      hasRectRadius(this.markConfigs[id]) &&
       graphic.items?.every(item => ["rect", "path"].includes(item.type));
     if (
       (graphic?.type !== "rect" && !roundedCollection) ||