Clean up Object Vector Studio icon assets layout spacing tests and geometry headers - PR_26133_022-object-vector-studio-layout-icons-and-test-cleanup

Author: DavidQ

…io-v2/starter-project-template/README.md …io-v2/starter-project-template/README.mdtools/object-vector-studio-v2/starter-project-template/README.md renamed to docs/dev/archive/object-vector-studio-v2/starter-project-template/README.md tools/object-vector-studio-v2/starter-project-template/README.md renamed to docs/dev/archive/object-vector-studio-v2/starter-project-template/README.md

@@ -0,0 +1,18 @@
+# PR_26133_022 Starter Template Report
+
+## Inspection
+
+- Inspected `tools/object-vector-studio-v2/starter-project-template`.
+- Searched active tool, test, script, package, and docs/dev report paths for references to the active path.
+- No active references to `tools/object-vector-studio-v2/starter-project-template` were found.
+
+## Result
+
+- Moved the unused template out of the active Object Vector Studio V2 tool path.
+- Archived location: `docs/dev/archive/object-vector-studio-v2/starter-project-template`.
+- The template was not deleted silently.
+
+## Boundary Checks
+
+- `src/shared/font/0xProtoNerdFont` was not deleted, moved, or modified.
+- No sample JSON files were changed.

…ect-template/config/starter.project.json …ect-template/config/starter.project.jsontools/object-vector-studio-v2/starter-project-template/config/starter.project.json renamed to docs/dev/archive/object-vector-studio-v2/starter-project-template/config/starter.project.json tools/object-vector-studio-v2/starter-project-template/config/starter.project.json renamed to docs/dev/archive/object-vector-studio-v2/starter-project-template/config/starter.project.json

@@ -1,4 +1,4 @@
-# PR_26133_021 Playwright V8 Coverage Report
+# PR_26133_022 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,16 +7,18 @@ 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%) Workspace Manager V2 - exercised 10 runtime JS files`.
 - Changed Object Vector Studio V2 runtime JS coverage from the generated report:
-  - `(83%) tools/object-vector-studio-v2/js/bootstrap.js - executed lines 106/106; executed functions 5/6`
-  - `(93%) tools/object-vector-studio-v2/js/ToolStarterApp.js - executed lines 4092/4092; executed functions 442/476`
+  - `(83%) tools/object-vector-studio-v2/js/bootstrap.js - executed lines 107/107; executed functions 5/6`
+  - `(93%) tools/object-vector-studio-v2/js/ToolStarterApp.js - executed lines 4096/4096; executed functions 443/477`
 - Changed JS files considered by the generated report:
   - `tools/object-vector-studio-v2/js/bootstrap.js`: covered by browser V8 coverage.
   - `tools/object-vector-studio-v2/js/ToolStarterApp.js`: covered by browser V8 coverage.
+  - `tools/object-vector-studio-v2/playwright.config.mjs`: not collected as browser runtime coverage.
+  - `tests/playwright/tools/ObjectVectorStudioV2FirstClassToolStarter.spec.mjs`: not collected as browser runtime coverage.
   - `tests/playwright/tools/WorkspaceManagerV2.spec.mjs`: not collected as browser runtime coverage.
 
 ## Validation Context
 
 - Main command: `npm run test:workspace-v2`.
 - Result: 48 passed.
-- Focused Object Vector Studio V2 palette, icon mapping, Object Geometry header, polygon checkbox add/delete, transform icon, geometry-layout, preview coordinate, mouse-editing, animation-state, and asset-library scenarios passed as part of the workspace-v2 run.
-- Coverage includes the PR21 Object Vector Studio V2 runtime changes for glyph mapping, Object Geometry header state, and polygon point controls.
+- Focused Object Vector Studio V2 scenarios covered the Nerd Font icon mapping cleanup, tighter accordion spacing, Object Geometry header layout, Object Transform summary placement, palette same-line controls, polygon Add Point behavior, preview coordinate/grid behavior, mouse editing, animation states, and asset-library scenarios.
+- The required moved spec also passed through `npx playwright test --config=tools/object-vector-studio-v2/playwright.config.mjs --workers=1 --reporter=list`.

docs/dev/reports/object_vector_studio_starter_template_report.md

@@ -1,24 +1,27 @@
-# PR_26133_021 Workspace V2 Results
+# PR_26133_022 Workspace V2 Results
 
 ## Command Results
 
 - `node --check tools/object-vector-studio-v2/js/ToolStarterApp.js`: passed.
 - `node --check tools/object-vector-studio-v2/js/bootstrap.js`: passed.
+- `node --check tests/playwright/tools/ObjectVectorStudioV2FirstClassToolStarter.spec.mjs`: passed.
 - `node --check tests/playwright/tools/WorkspaceManagerV2.spec.mjs`: passed.
-- `npx playwright test tests/playwright/tools/WorkspaceManagerV2.spec.mjs --project=playwright --workers=1 --reporter=list -g "shows Object Vector Studio V2 layout shell and schema-only palette gate"`: 1 passed.
+- `npx playwright test --config=tools/object-vector-studio-v2/playwright.config.mjs --workers=1 --reporter=list`: 4 passed.
 - `npm run test:workspace-v2`: 48 passed.
-- `git diff --check`: passed with LF-to-CRLF working-copy warnings for touched files.
+- `git diff --check`: passed with LF-to-CRLF working-copy warnings only.
 
 ## Targeted Object Vector Studio V2 Verification
 
-- Confirmed Palette no longer renders a `Stroke Width` label; `Width` sits to the right of the Stroke button in the palette control row.
-- Confirmed Shape/Tools icons use the requested mappings for Angle Snap, Select, Triangle, Rectangle, and existing tool glyphs; Select/Triangle/Rectangle are enlarged and Rectangle is offset upward.
-- Confirmed Object Transform action buttons use `nf-fa-scale_unbalanced` for Scale and `nf-md-resize` for Resize.
-- Confirmed Object Details is renamed to Object Geometry and the selected shape summary appears in the accordion header while the body no longer repeats `Shape <id> (<type>)`.
-- Confirmed polygon point rows expose checkboxes, Add Side inserts after the checked point and clears selections, Delete Point removes checked rows and clears selections, and invalid deletes are visibly rejected before schema-invalid point counts can apply.
-- Confirmed Object Add/Rename/Dup controls render directly under Object Name and above the Tag line.
-- Confirmed Object Vector Studio V2 tests reported no console/page errors in the targeted browser scenarios.
-- Confirmed `src/shared/font/0xProtoNerdFont` was preserved.
+- Confirmed old pre-Nerd Font CSS icon drawings were removed from active Object Vector Studio V2 icon classes while `src/shared/font/0xProtoNerdFont` remained untouched.
+- Confirmed Grid Snap uses `nf-md-vector_point`; Select, Triangle, and Line use the requested icon offsets/sizing adjustments.
+- Confirmed Object Vector Studio V2 accordion spacing is tighter through reduced panel gap, header height/padding, content padding, and status header height.
+- Confirmed the moved Playwright spec runs from `tests/playwright/tools/ObjectVectorStudioV2FirstClassToolStarter.spec.mjs` through the updated tool config.
+- Confirmed the unused starter template was moved to `docs/dev/archive/object-vector-studio-v2/starter-project-template` and documented in `object_vector_studio_starter_template_report.md`.
+- Confirmed Object Transform no longer renders `Selected Shape: <id>` and places the live transform summary below the transform action buttons.
+- Confirmed Add Tag icon sizing matches Add Object, and polygon `Add Side` is renamed to `Add Point`.
+- Confirmed Palette Paint, Stroke, and Width controls render on one line with a compact Width input.
+- Confirmed Object Geometry header renders as `Object Geometry (type)` on the first line and the selected shape id on the next line.
+- Confirmed Object Vector Studio V2 targeted browser scenarios reported no console/page errors.
 
 ## Scope Checks
 

docs/dev/reports/playwright_v8_coverage_report.md

@@ -4,7 +4,7 @@ import { extname, join, normalize, relative, resolve } from "node:path";
 import { readFile, stat, writeFile } from "node:fs/promises";
 import { fileURLToPath } from "node:url";
 
-const REPO_ROOT = resolve(fileURLToPath(new URL("../../../..", import.meta.url)));
+const REPO_ROOT = resolve(fileURLToPath(new URL("../../..", import.meta.url)));
 const CONTENT_TYPES = new Map([
   [".css", "text/css; charset=utf-8"],
   [".html", "text/html; charset=utf-8"],
@@ -71,16 +71,17 @@ test.describe("Object Vector Studio V2", () => {
     await expect(page.locator("#objectVectorStudioV2CopyJsonButton")).toBeDisabled();
     await expect(page.locator("#objectVectorStudioV2ExportJsonButton")).toBeDisabled();
     await expect(page.locator("#objectVectorStudioV2PaletteSwatchCount")).toHaveText("(0 swatches)");
-    await expect(page.locator("#objectVectorStudioV2ObjectDetailsCount")).toHaveText("(0 obj, 0 shapes)");
+    await expect(page.locator("#objectVectorStudioV2ObjectsCount")).toHaveText("(0 obj, 0 shapes)");
     await expect(page.locator("#objectVectorStudioV2ObjectTiles")).toContainText("No objects loaded");
     await expect(page.locator("#objectVectorStudioV2RenameObjectButton")).toBeDisabled();
-    await expect(page.locator("#objectVectorStudioV2DeleteObjectButton")).toBeDisabled();
+    await expect(page.locator("#objectVectorStudioV2ObjectTransform")).toHaveText("No shape selected.");
     await expect(page.locator("#objectVectorStudioV2FlattenObjectButton")).toHaveCount(0);
     await expect(page.getByRole("button", { name: "Object", exact: true })).toBeVisible();
     await expect(page.getByRole("button", { name: "Shape/Tools", exact: true })).toBeVisible();
-    await expect(page.getByRole("button", { name: "Objects", exact: true })).toBeVisible();
+    await expect(page.getByRole("button", { name: "Objects (0 obj, 0 shapes)" })).toBeVisible();
     await expect(page.getByRole("button", { name: "Palette (0 swatches)" })).toBeVisible();
-    await expect(page.getByRole("button", { name: "Object Details (0 obj, 0 shapes)" })).toBeVisible();
+    await expect(page.getByRole("button", { name: "Object Geometry", exact: true })).toBeVisible();
+    await expect(page.getByRole("button", { name: "Object Transform", exact: true })).toBeVisible();
     await expect(page.getByRole("button", { name: "JSON Details" })).toBeVisible();
     await expect(page.locator("#statusLog")).toHaveValue(/Object Vector Studio V2 layout shell ready\./);
     await expect(page.locator("#statusLog")).toHaveValue(/INFO Shape\/Tools primitive buttons create schema-valid shapes on the selected object\./);
@@ -125,17 +126,17 @@ test.describe("Object Vector Studio V2", () => {
 
     await page.locator("#objectVectorStudioV2ObjectNameInput").fill("Local Object");
     await page.locator("#objectVectorStudioV2AddObjectButton").click();
-    await expect(page.locator("#objectVectorStudioV2ObjectDetailsCount")).toHaveText("(3 obj, 0 shapes)");
+    await expect(page.locator("#objectVectorStudioV2ObjectsCount")).toHaveText("(3 obj, 0 shapes)");
     await expect(page.locator('[data-object-id="object.local.local-object"]')).toHaveAttribute("aria-pressed", "true");
     await expect(page.locator("#objectVectorStudioV2JsonDetails")).toContainText('"name": "Local Object"');
 
     await page.locator('[data-shape-tool="rectangle"]').click();
-    await expect(page.locator("#objectVectorStudioV2ObjectDetailsCount")).toHaveText("(3 obj, 1 shape)");
+    await expect(page.locator("#objectVectorStudioV2ObjectsCount")).toHaveText("(3 obj, 1 shape)");
     await expect(page.locator("#objectVectorStudioV2RenderSurface [data-shape-id='rectangle-1']")).toHaveClass(/is-selected/);
     await expect(page.locator("#statusLog")).toHaveValue(/OK Created rectangle shape rectangle-1 on Local Object\./);
 
-    await page.locator("#objectVectorStudioV2DeleteObjectButton").click();
-    await expect(page.locator("#objectVectorStudioV2ObjectDetailsCount")).toHaveText("(2 obj, 0 shapes)");
+    await page.locator('[data-object-id="object.local.local-object"] [data-object-control="delete"]').click();
+    await expect(page.locator("#objectVectorStudioV2ObjectsCount")).toHaveText("(2 obj, 0 shapes)");
     await expect(page.locator('[data-object-id="object.local.local-object"]')).toHaveCount(0);
   });