Normalize Object Preview coordinate display without changing visual grid scale - PR_26133_005-object-preview-coordinate-display-normalization

Author: DavidQ

docs/dev/reports/playwright_v8_coverage_report.md

@@ -1,4 +1,4 @@
-# PR_26133_004 Playwright V8 Coverage Report
+# PR_26133_005 Playwright V8 Coverage Report
 
 Coverage source: `docs/dev/reports/playwright_v8_coverage_report.txt`, refreshed by the final `npm run test:workspace-v2` run.
 
@@ -7,7 +7,7 @@ Coverage source: `docs/dev/reports/playwright_v8_coverage_report.txt`, refreshed
 - Coverage is advisory only; no thresholds are enforced.
 - Workspace Manager V2 entry point: 91%.
 - Object Vector Studio V2 runtime coverage entries from the generated report:
-  - `tools/object-vector-studio-v2/js/ToolStarterApp.js`: 91%, executed lines 3304/3304, executed functions 346/381.
+  - `tools/object-vector-studio-v2/js/ToolStarterApp.js`: 91%, executed lines 3313/3313, executed functions 349/384.
   - `tools/object-vector-studio-v2/js/bootstrap.js`: 80%, executed lines 99/99, executed functions 4/5.
 - The generated report lists `tools/object-vector-studio-v2/js/ToolStarterApp.js` as a changed JS file with browser V8 coverage.
 
@@ -16,4 +16,4 @@ Coverage source: `docs/dev/reports/playwright_v8_coverage_report.txt`, refreshed
 - Main command: `npm run test:workspace-v2`.
 - Result: 46 passed.
 - Focused Object Vector Studio V2 layout and preview-coordinate scenarios also passed.
-- Manual Object Preview probe confirmed restored 10-unit grid spacing, 10x object drawing scale, zoom/pan/reset behavior, and no console/runtime errors.
+- Manual Object Preview probe confirmed logical pointer/origin display, unchanged visual scale, and no console/runtime errors.

docs/dev/reports/playwright_workspace_v2_results.md

@@ -1,27 +1,27 @@
-# PR_26133_004 Workspace V2 Results
+# PR_26133_005 Workspace V2 Results
 
 ## Command Results
 
 - `node --check tools/object-vector-studio-v2/js/ToolStarterApp.js`: passed.
 - `node --check tests/playwright/tools/WorkspaceManagerV2.spec.mjs`: passed.
-- `npx playwright test tests/playwright/tools/WorkspaceManagerV2.spec.mjs --grep "Object Vector Studio V2 (layout shell|preview coordinates)"`: 2 passed.
+- `npx playwright test tests/playwright/tools/WorkspaceManagerV2.spec.mjs --grep "Object Vector Studio V2 (layout shell|preview coordinates)"`: 2 passed after correcting the pointer-event test probe.
 - `npm run test:workspace-v2`: 46 passed.
 - `git diff --check`: passed with the existing LF-to-CRLF working-copy warning for `tests/playwright/tools/WorkspaceManagerV2.spec.mjs`.
 
 ## Targeted Object Preview Verification
 
 - Standalone Playwright manual probe reported `consoleErrors: []` and `pageErrors: []`.
-- Object Preview grid rendered with 10-unit visible spacing.
-- Object Preview shape drawing used 10x preview scale while preserving persisted object coordinates.
-- Asteroids ship source points `0,-18`, `14,16`, `0,8`, `-14,16` rendered as `0,-180`, `140,160`, `0,80`, `-140,160`.
-- At 25% zoom, the Asteroids ship measured 18 visible grid lines above origin and 16 visible grid lines below origin.
-- Object Preview canvas filled available horizontal work-area width and kept the SVG viewBox aspect ratio stable.
-- Zoom, pan, and reset view remained functional: panned viewBox `-620 -440 1280 880`, reset viewBox `-160 -110 320 220`.
+- Object Preview visuals remained unchanged: grid step `10`, viewBox `-640 -440 1280 880` at 25% zoom, and Asteroids ship drawn points `0,-180`, `140,160`, `0,80`, `-140,160`.
+- Pointer display now reports logical coordinates: grid-space `-140,-160` displays as `Pointer -14, -16 | Canvas origin 0,0 centered | Zoom 25%`.
+- Canvas origin display now reports logical coordinates: panning right displays `Origin: 2, 0 | Canvas 0,0 centered | Zoom 25%`.
+- Reset view displays `Origin: 0, 0 | Canvas 0,0 centered | Zoom 100%`.
+- Zoom display remains logical viewport zoom and is not multiplied by `GRID_STEP`.
 
 ## Contract Checks
 
-- `const GRID_STEP = 10;` is restored in Object Preview.
-- Canvas side-to-side fit and aspect-ratio behavior from the prior PR are preserved.
-- No sample JSON files were modified.
+- `const GRID_STEP = 10;` remains unchanged.
+- Object rendering scale was not changed.
+- Grid rendering was not changed.
+- Canvas sizing was not changed.
 - No unrelated tool/runtime files were changed.
 - No fallback behavior was added.

tests/playwright/tools/WorkspaceManagerV2.spec.mjs

@@ -1842,9 +1842,10 @@ test.describe("Workspace Manager V2 bootstrap", () => {
       await expect(page.locator("#objectVectorStudioV2RenderSurface")).toHaveAttribute("viewBox", "-145.455 -100 290.909 200");
       await page.locator("#objectVectorStudioV2PanRightButton").click();
       await expect(page.locator("#objectVectorStudioV2RenderSurface")).toHaveAttribute("viewBox", "-125.455 -100 290.909 200");
+      await expect(page.locator("#objectVectorStudioV2CoordinateDisplay")).toHaveText("Origin: 2, 0 | Canvas 0,0 centered | Zoom 110%");
       await page.locator("#objectVectorStudioV2ResetViewButton").click();
       await expect(page.locator("#objectVectorStudioV2RenderSurface")).toHaveAttribute("viewBox", "-160 -110 320 220");
-      await expect(page.locator("#objectVectorStudioV2CoordinateDisplay")).toContainText("Origin: 0, 0");
+      await expect(page.locator("#objectVectorStudioV2CoordinateDisplay")).toHaveText("Origin: 0, 0 | Canvas 0,0 centered | Zoom 100%");
       await expect(page.locator("#statusLog")).toHaveValue(/OK Viewport reset to 100% at origin 0,0\./);
       await page.locator("#objectVectorStudioV2RenderSurface").hover();
       for (let index = 0; index < 40; index += 1) {
@@ -2206,8 +2207,23 @@ test.describe("Workspace Manager V2 bootstrap", () => {
 
       await page.locator("#objectVectorStudioV2PanRightButton").click();
       await expect(page.locator("#objectVectorStudioV2RenderSurface")).toHaveAttribute("viewBox", "-620 -440 1280 880");
+      await expect(page.locator("#objectVectorStudioV2CoordinateDisplay")).toHaveText("Origin: 2, 0 | Canvas 0,0 centered | Zoom 25%");
       await page.locator("#objectVectorStudioV2PanLeftButton").click();
       await expect(page.locator("#objectVectorStudioV2RenderSurface")).toHaveAttribute("viewBox", "-640 -440 1280 880");
+      await expect(page.locator("#objectVectorStudioV2CoordinateDisplay")).toHaveText("Origin: 0, 0 | Canvas 0,0 centered | Zoom 25%");
+      const examplePointerScreen = await page.locator("#objectVectorStudioV2RenderSurface").evaluate((surface) => {
+        const point = surface.createSVGPoint();
+        point.x = -140;
+        point.y = -160;
+        const transformed = point.matrixTransform(surface.getScreenCTM());
+        return { x: transformed.x, y: transformed.y };
+      });
+      await page.locator("#objectVectorStudioV2RenderSurface").dispatchEvent("mousemove", {
+        bubbles: true,
+        clientX: examplePointerScreen.x,
+        clientY: examplePointerScreen.y
+      });
+      await expect(page.locator("#objectVectorStudioV2CoordinateDisplay")).toHaveText("Pointer -14, -16 | Canvas origin 0,0 centered | Zoom 25%");
       await page.setViewportSize({ width: 1040, height: 720 });
       const resizedPreviewScale = await readPreviewScale();
       expect(resizedPreviewScale.aspectRatioStable).toBe(true);

tools/object-vector-studio-v2/js/ToolStarterApp.js

@@ -1867,14 +1867,26 @@ export class ToolStarterApp {
     const viewX = this.formatViewportNumber(this.viewport.x - width / 2);
     const viewY = this.formatViewportNumber(this.viewport.y - height / 2);
     this.elements.renderSurface.setAttribute("viewBox", `${viewX} ${viewY} ${this.formatViewportNumber(width)} ${this.formatViewportNumber(height)}`);
-    this.elements.coordinateDisplay.textContent = `Origin: ${this.formatViewportNumber(this.viewport.x)}, ${this.formatViewportNumber(this.viewport.y)} | Canvas 0,0 centered | Zoom ${Math.round(this.viewport.zoom * 100)}%`;
+    this.elements.coordinateDisplay.textContent = `Origin: ${this.formatLogicalCoordinate(this.viewport.x)}, ${this.formatLogicalCoordinate(this.viewport.y)} | Canvas 0,0 centered | Zoom ${this.formatZoomPercentage()}%`;
   }
 
   formatViewportNumber(value) {
     const normalized = Number(Number(value).toFixed(3));
     return Object.is(normalized, -0) ? 0 : normalized;
   }
 
+  formatLogicalCoordinate(value) {
+    return this.formatViewportNumber(value / GRID_STEP);
+  }
+
+  formatLogicalPointerCoordinate(value) {
+    return Math.round(this.formatLogicalCoordinate(value));
+  }
+
+  formatZoomPercentage() {
+    return Math.round(this.viewport.zoom * 100);
+  }
+
   zoomViewportByStep(step) {
     const nextZoom = Number((this.viewport.zoom + step).toFixed(3));
     this.zoomViewport(nextZoom);
@@ -1913,7 +1925,7 @@ export class ToolStarterApp {
     const viewHeight = DEFAULT_VIEWPORT.height / this.viewport.zoom;
     const x = this.viewport.x - viewWidth / 2 + ((event.clientX - bounds.left) / bounds.width) * viewWidth;
     const y = this.viewport.y - viewHeight / 2 + ((event.clientY - bounds.top) / bounds.height) * viewHeight;
-    this.elements.coordinateDisplay.textContent = `Pointer ${Math.round(x)}, ${Math.round(y)} | Canvas origin 0,0 centered | Zoom ${Math.round(this.viewport.zoom * 100)}%`;
+    this.elements.coordinateDisplay.textContent = `Pointer ${this.formatLogicalPointerCoordinate(x)}, ${this.formatLogicalPointerCoordinate(y)} | Canvas origin 0,0 centered | Zoom ${this.formatZoomPercentage()}%`;
   }
 
   selectObject(objectId, sourceLabel) {