@orbat-mapper/tactical-draw 0.10.0 → 0.10.2

package/dist/index.d.mts

@@ -1662,6 +1662,12 @@ declare class TacticalDraw {
   private transformCounter;
   /** Snapshot of the host's latest authoritative Graphic membership and order. */
   private lastRenderedMeasures;
+  /**
+   * Id index over {@link lastRenderedMeasures}, rebuilt alongside it so
+   * per-id lookups (e.g. {@link renderHighlights}) are O(1) instead of an
+   * O(document) linear scan on every highlighted id, every frame.
+   */
+  private lastRenderedMeasuresById;
   /** Whether any last-rendered graphic reads the whole camera, not just scale. */
   private lastRenderedHasViewDependent;
   private snapGeometryFeatures;
@@ -1703,6 +1709,15 @@ declare class TacticalDraw {
    * that removes outlines when the set transitions to empty.
    */
   private hasHighlightOutlines;
+  /**
+   * Coalesces bursts of {@link setHighlightedGraphics} (e.g. hover across a
+   * list of rows) into at most one {@link renderHighlights} per animation
+   * frame. Every other caller — `render()`, view-change, session start /
+   * settle — needs highlight state updated synchronously and in step with
+   * its own write, so those call {@link flushHighlights} instead, which
+   * cancels any pending coalesced frame before rendering immediately.
+   */
+  private readonly highlightCoalescer;
   constructor(adapter: MapAdapter, options?: TacticalDrawOptions);
   setPinPresentation(defaults: PinPresentationPatch): void;
   setPinOverride(id: string, patch: PinOverridePatch): void;
@@ -1830,6 +1845,14 @@ declare class TacticalDraw {
    * "locked selection" chrome.
    */
   setHighlightedGraphics(ids: readonly string[]): void;
+  /**
+   * Cancel any pending coalesced highlight frame and re-project highlights
+   * immediately. Used by every caller other than `setHighlightedGraphics`
+   * (`render()`, view-change, session start / settle) so their own
+   * synchronous effects and the highlight layer update together, with no
+   * frame of lag and no stale coalesced call left to fire afterwards.
+   */
+  private flushHighlights;
   /**
    * Project the current {@link highlightedIds} into one outline `LineString`
    * per graphic and reconcile them onto the highlight layer. Idempotent and

package/dist/index.mjs

@@ -7644,6 +7644,12 @@ var TacticalDraw = class {
 	transformCounter = 0;
 	/** Snapshot of the host's latest authoritative Graphic membership and order. */
 	lastRenderedMeasures = [];
+	/**
+	* Id index over {@link lastRenderedMeasures}, rebuilt alongside it so
+	* per-id lookups (e.g. {@link renderHighlights}) are O(1) instead of an
+	* O(document) linear scan on every highlighted id, every frame.
+	*/
+	lastRenderedMeasuresById = /* @__PURE__ */ new Map();
 	/** Whether any last-rendered graphic reads the whole camera, not just scale. */
 	lastRenderedHasViewDependent = false;
 	snapGeometryFeatures = [];
@@ -7685,6 +7691,15 @@ var TacticalDraw = class {
 	* that removes outlines when the set transitions to empty.
 	*/
 	hasHighlightOutlines = false;
+	/**
+	* Coalesces bursts of {@link setHighlightedGraphics} (e.g. hover across a
+	* list of rows) into at most one {@link renderHighlights} per animation
+	* frame. Every other caller — `render()`, view-change, session start /
+	* settle — needs highlight state updated synchronously and in step with
+	* its own write, so those call {@link flushHighlights} instead, which
+	* cancels any pending coalesced frame before rendering immediately.
+	*/
+	highlightCoalescer = createFrameCoalescer(() => this.renderHighlights());
 	constructor(adapter, options) {
 		this.adapter = adapter;
 		this.offsetDragModifier = options?.offsetDragModifier ?? false;
@@ -7717,7 +7732,7 @@ var TacticalDraw = class {
 			this.invalidateGraphicGroupCache(false);
 			this.refreshActiveEditPresentation();
 			this.reconcileGraphicsStack();
-			this.renderHighlights();
+			this.flushHighlights();
 		});
 	}
 	setPinPresentation(defaults) {
@@ -7951,7 +7966,7 @@ var TacticalDraw = class {
 		const resolutionChanged = !Object.is(resolution, this.lastViewResolution);
 		this.lastViewResolution = resolution;
 		if (!this.hasRendered) return;
-		this.renderHighlights();
+		this.flushHighlights();
 		const nativePointSymbolScaling = this.adapter.scalesGroundPointSymbolsNatively === true;
 		if (!resolutionChanged && !this.lastRenderedHasViewDependent) return;
 		const excluded = this.graphicsStackExcludedIds();
@@ -8040,6 +8055,7 @@ var TacticalDraw = class {
 			}
 		}
 		this.lastRenderedMeasures = measures.slice();
+		this.lastRenderedMeasuresById = new Map(this.lastRenderedMeasures.map((g) => [g.id, g]));
 		this.lastRenderedHasViewDependent = this.lastRenderedMeasures.some((g) => !this.renderGraphicGroup(g).pin && graphicViewDependence(g, this.adapter.scalesGroundPointSymbolsNatively === true) === "view");
 		this.graphicStack.acknowledgeHostRender(measures);
 		if (activeEdit && "syncHostMembership" in activeEdit && measures.some((graphic) => graphic.id === activeEdit.measureId && graphic.kind === activeEdit.graphicKind)) activeEdit.syncHostMembership(true);
@@ -8058,7 +8074,7 @@ var TacticalDraw = class {
 			});
 			else if (this.activeInteraction.isStarting) this.activeInteraction.cancel("removed");
 		}
-		this.renderHighlights();
+		this.flushHighlights();
 	}
 	/**
 	* Replace the passive selection-highlight set with `ids`. Declarative and
@@ -8073,6 +8089,17 @@ var TacticalDraw = class {
 	setHighlightedGraphics(ids) {
 		this.assertAlive();
 		this.highlightedIds = ids.slice();
+		this.highlightCoalescer.schedule();
+	}
+	/**
+	* Cancel any pending coalesced highlight frame and re-project highlights
+	* immediately. Used by every caller other than `setHighlightedGraphics`
+	* (`render()`, view-change, session start / settle) so their own
+	* synchronous effects and the highlight layer update together, with no
+	* frame of lag and no stale coalesced call left to fire afterwards.
+	*/
+	flushHighlights() {
+		this.highlightCoalescer.cancel();
 		this.renderHighlights();
 	}
 	/**
@@ -8098,7 +8125,7 @@ var TacticalDraw = class {
 		const pinGroups = [];
 		for (const id of this.highlightedIds) {
 			if (excluded.has(id)) continue;
-			const graphic = this.lastRenderedMeasures.find((g) => g.id === id);
+			const graphic = this.lastRenderedMeasuresById.get(id);
 			if (!graphic) continue;
 			if (isPointSymbol(graphic)) {
 				const pin = this.renderGraphicGroup(graphic).pin;
@@ -8151,16 +8178,13 @@ var TacticalDraw = class {
 			if (!this.images) return null;
 			return imageFootprint(graphic, renderImageForAdapter(this.adapter, this.images, graphic).render, this.adapter)?.box ?? null;
 		}
-		const render = renderMeasureForAdapter(this.adapter, graphic, {
-			graphicsStyle: this.graphicsStyle,
-			symbolColors: this.symbolColors
-		});
-		if (graphic.kind === "text") return fitTextFootprintFromFeatures(render.features, (lonLat) => this.adapter.getPixelFromCoordinate(lonLat), {
+		const features = this.renderGraphicGroup(graphic).features;
+		if (graphic.kind === "text") return fitTextFootprintFromFeatures(features, (lonLat) => this.adapter.getPixelFromCoordinate(lonLat), {
 			resolution: this.adapter.getResolution(),
 			bearing
 		});
 		const positions = [];
-		for (const feature of render.features) if (feature.geometry) collectFeaturePositions(feature.geometry, positions);
+		for (const feature of features) if (feature.geometry) collectFeaturePositions(feature.geometry, positions);
 		const pixels = [];
 		for (const position of positions) {
 			const pixel = this.adapter.getPixelFromCoordinate(position);
@@ -8221,7 +8245,7 @@ var TacticalDraw = class {
 			clearSlot();
 			this.clearSnapResult();
 			if (extraSettle) extraSettle();
-			this.renderHighlights();
+			this.flushHighlights();
 		};
 		this.activeInteraction.begin();
 		try {
@@ -8235,7 +8259,7 @@ var TacticalDraw = class {
 			this.activeInteraction.set(handle);
 			if (activate && !handle.settled) {
 				deactivate = activate(handle) ?? null;
-				this.renderHighlights();
+				this.flushHighlights();
 			}
 		}
 		handle.promise.then(clearSlot, clearSlot);
@@ -8782,6 +8806,7 @@ var TacticalDraw = class {
 		this.unsubscribePins();
 		if (this.destroyed) return;
 		this.destroyed = true;
+		this.highlightCoalescer.cancel();
 		if (this.activeInteraction.isActive) {
 			this.activeEdit = null;
 			this.activeTransform = null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orbat-mapper/tactical-draw",
-  "version": "0.10.0",
+  "version": "0.10.2",
   "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.27.0"
+    "@orbat-mapper/control-measures": "^0.28.0"
   },
   "peerDependencies": {
     "vitest": ">=4.0.0"