@orbat-mapper/tactical-draw 0.2.0-alpha.39 → 0.2.0-alpha.40

package/dist/index.d.mts

@@ -685,8 +685,18 @@ interface EditSession {
    * and on `close()`. A no-op for measures with no pixel-denominated size. The
    * in-flight preview is unaffected (it renders from the working options), so a
    * screen-locked symbol stays screen-locked until commit.
-   */
-  setSizeAnchor(anchor: SizeAnchor): void;
+   *
+   * `options` shallow-merges a size-option patch in the *same* mutation, as
+   * [[setOptions]] would. Changing a measure's size mode means moving its value
+   * between the pixel and meter halves of a size pair *and* re-anchoring, and
+   * the halves are only meaningful together: options carrying a pixel size
+   * under a `"ground"` anchor bake that size away on commit, so a host that
+   * emitted the two as separate mutations would leave an undo step parked on a
+   * state whose committed size mode contradicts its own options. Passed here
+   * they are one change event and one history entry, which undo cannot land
+   * inside.
+   */
+  setSizeAnchor(anchor: SizeAnchor, options?: ControlMeasure["options"]): void;
   /** Shallow-merge per-measure style into the live working copy. */
   setStyle(partial: ControlMeasureStyle): void;
   /**

package/dist/index.mjs

@@ -4322,11 +4322,24 @@ function runEdit(opts) {
 	* and emits a change so hosts see the re-anchored measure. The working render
 	* is unaffected — it renders from `workingOptions`, which still carry the
 	* pixel size, so the in-flight symbol stays screen-locked until commit.
+	*
+	* An `options` patch rides along in the same mutation, merged as
+	* [[setOptions]] merges. That pairing is the point: a size *mode* switch
+	* moves the value between a size pair's pixel and meter halves and re-anchors
+	* the measure, and only the combination is coherent — pixel options under a
+	* `"ground"` anchor bake back to meters on commit. As two mutations they
+	* would be two history entries with exactly that incoherent state between
+	* them, reachable by a single undo.
 	*/
-	function setSizeAnchor(next) {
-		if (ctx.settled || sizeAnchor === next) return;
-		const previousPoints = workingPoints.map((p) => [...p]);
+	function setSizeAnchor(next, options) {
+		if (ctx.settled) return;
 		const previousOptions = { ...workingOptions ?? {} };
+		const nextOptions = options ? {
+			...previousOptions,
+			...options
+		} : void 0;
+		if (sizeAnchor === next && (!nextOptions || jsonEqual(nextOptions, previousOptions))) return;
+		const previousPoints = workingPoints.map((p) => [...p]);
 		const previousStyle = currentStyle;
 		const previousNonGeometryDirty = nonGeometryDirty;
 		const previousSizeAnchor = sizeAnchor;
@@ -4335,10 +4348,19 @@ function runEdit(opts) {
 		const previousProperties = snapshotProperties();
 		const previousRendered = rebuildRendered();
 		sizeAnchor = next;
-		const currentRendered = rebuildRendered();
+		if (nextOptions) replaceWorkingOptions(nextOptions);
+		let currentRendered;
+		try {
+			currentRendered = rebuildRendered();
+		} catch (err) {
+			sizeAnchor = previousSizeAnchor;
+			replaceWorkingOptions(previousOptions);
+			throw err;
+		}
+		if (nextOptions) renderWorkingGraphicAndHandles();
 		nonGeometryDirty = true;
 		dirty = true;
-		emitChange(previousRendered, previousPoints, previousOptions, previousStyle, previousNonGeometryDirty, previousSizeAnchor, currentRendered, previousTextAmplifiers, previousAmplifierPlacements, previousProperties, "programmatic", "Change size anchor");
+		emitChange(previousRendered, previousPoints, previousOptions, previousStyle, previousNonGeometryDirty, previousSizeAnchor, currentRendered, previousTextAmplifiers, previousAmplifierPlacements, previousProperties, "programmatic", nextOptions ? "Change size mode" : "Change size anchor");
 	}
 	function modesEqual(a, b) {
 		if (a.length !== b.length) return false;
@@ -4473,8 +4495,8 @@ function runEdit(opts) {
 		setAmplifierPlacements(partial) {
 			setAmplifierPlacements(partial);
 		},
-		setSizeAnchor(next) {
-			setSizeAnchor(next);
+		setSizeAnchor(next, options) {
+			setSizeAnchor(next, options);
 		},
 		onChange(listener) {
 			if (ctx.settled) return () => {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orbat-mapper/tactical-draw",
-  "version": "0.2.0-alpha.39",
+  "version": "0.2.0-alpha.40",
   "description": "Engine-agnostic draw/edit core for tactical control measures (MIL-STD-2525 / APP-6). The MapAdapter ABI and bring-your-own-engine entry point for the tactical-draw package family.",
   "license": "MIT",
   "author": "Orbat Mapper",
@@ -39,7 +39,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@orbat-mapper/control-measures": "^0.2.0-alpha.28"
+    "@orbat-mapper/control-measures": "^0.2.0-alpha.30"
   },
   "peerDependencies": {
     "vitest": ">=4.0.0"