Fix Asset Manager V2 history accordion and control spacing - PR_26126_099-asset-manager-v2-accordion-spacing-fix

Author: DavidQ

docs/dev/reports/PR_26126_099_asset_manager_v2_accordion_spacing_notes.md

@@ -0,0 +1,21 @@
+# PR_26126_099 Asset Manager V2 Accordion Spacing Notes
+
+## History Accordion
+- History now uses the shared `accordion-v2` header/body markup with `assetHistoryContent`.
+- The History header is bound by the existing Asset Manager V2 accordion controller.
+- Collapsing History hides the body, updates `aria-expanded`, updates the icon state, and reduces the accordion section height.
+- History remains outside Asset Controls and directly under the Asset Controls accordion in the left panel.
+
+## Spacing
+- Open accordions now size to content by default instead of flexing to fill all available panel height.
+- Output Summary and Status keep explicit fill behavior where their panel layout depends on it.
+- Asset Controls, Assets, and Selected Asset Detail no longer carry measurable trailing body gap after their final visible child.
+
+## Scope Guard
+- No asset schema files were changed.
+- Import/export behavior was not changed.
+- Asset path handling was not changed.
+- Sample JSON files were not modified.
+
+## Validation
+- `npm run test:workspace-v2` validates History collapse/expand behavior and trailing spacing for Asset Controls, Assets, and Selected Asset Detail.

docs/dev/reports/PR_26126_099_asset_manager_v2_manual_validation_notes.md

@@ -0,0 +1,17 @@
+# PR_26126_099 Asset Manager V2 Manual Validation Notes
+
+## Commands
+- `npm run test:workspace-v2`
+- `git diff --check`
+
+## Results
+- Required workspace-v2 Playwright gate passed: 12 tests passed.
+- `docs/dev/reports/playwright_v8_coverage_report.txt` was regenerated by the required workspace-v2 validation run.
+- `git diff --check` passed with no whitespace errors.
+
+## Manual Checks Covered By Tests
+- History renders as an Asset Manager V2 accordion with the `History` header.
+- History collapse/expand updates visibility, `aria-expanded`, icon state, and released vertical height.
+- History remains directly under Asset Controls and outside the Asset Controls body.
+- Asset Controls, Assets, and Selected Asset Detail do not leave extra bottom whitespace after their final visible body content.
+- Asset schema, import/export behavior, path handling, and sample JSON are unchanged by this PR.

docs/dev/reports/codex_changed_files.txt

@@ -2,31 +2,19 @@
 M docs/dev/reports/coverage_changed_js_guardrail.txt
  M docs/dev/reports/playwright_v8_coverage_report.txt
  M tests/playwright/PreviewGeneratorV2Baseline.spec.mjs
- M tools/asset-manager-v2/README.md
  M tools/asset-manager-v2/index.html
- M tools/asset-manager-v2/js/AssetManagerV2App.js
- M tools/asset-manager-v2/js/assetManagerMetadata.js
- M tools/asset-manager-v2/js/bootstrap.js
- M tools/asset-manager-v2/js/controls/ActionNavControl.js
  M tools/asset-manager-v2/styles/assetManager.css
-?? docs/dev/reports/PR_26126_098_asset_manager_v2_manual_validation_notes.md
-?? docs/dev/reports/PR_26126_098_asset_manager_v2_nav_import_export_notes.md
-?? docs/dev/reports/PR_26126_098_asset_manager_v2_path_preservation_notes.md
+?? docs/dev/reports/PR_26126_099_asset_manager_v2_accordion_spacing_notes.md
+?? docs/dev/reports/PR_26126_099_asset_manager_v2_manual_validation_notes.md
 
 # git ls-files --others --exclude-standard
-docs/dev/reports/PR_26126_098_asset_manager_v2_manual_validation_notes.md
-docs/dev/reports/PR_26126_098_asset_manager_v2_nav_import_export_notes.md
-docs/dev/reports/PR_26126_098_asset_manager_v2_path_preservation_notes.md
+docs/dev/reports/PR_26126_099_asset_manager_v2_accordion_spacing_notes.md
+docs/dev/reports/PR_26126_099_asset_manager_v2_manual_validation_notes.md
 
 # git diff --stat
-docs/dev/reports/coverage_changed_js_guardrail.txt |   7 +-
- docs/dev/reports/playwright_v8_coverage_report.txt |  22 ++--
- .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 119 +++++++++++++++++++--
- tools/asset-manager-v2/README.md                   |   2 +-
- tools/asset-manager-v2/index.html                  |  16 +--
- tools/asset-manager-v2/js/AssetManagerV2App.js     |  71 ++++++++++--
- tools/asset-manager-v2/js/assetManagerMetadata.js  |   6 +-
- tools/asset-manager-v2/js/bootstrap.js             |   6 +-
- .../js/controls/ActionNavControl.js                |  32 +++---
- tools/asset-manager-v2/styles/assetManager.css     |   5 +-
- 10 files changed, 230 insertions(+), 56 deletions(-)
+docs/dev/reports/coverage_changed_js_guardrail.txt |  2 --
+ docs/dev/reports/playwright_v8_coverage_report.txt |  4 ---
+ .../playwright/PreviewGeneratorV2Baseline.spec.mjs | 38 +++++++++++++++++++---
+ tools/asset-manager-v2/index.html                  | 17 +++++++---
+ tools/asset-manager-v2/styles/assetManager.css     | 18 ++++++----
+ 5 files changed, 57 insertions(+), 22 deletions(-)