Skip to content

Stabilize GridLayout interactions and component coverage - #2341

Open
heswell wants to merge 12 commits into
mainfrom
heswell-copilot-finish-grid-layout-tests
Open

Stabilize GridLayout interactions and component coverage#2341
heswell wants to merge 12 commits into
mainfrom
heswell-copilot-finish-grid-layout-tests

Conversation

@heswell

@heswell heswell commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replays the gridlayout-tabs foundation onto current main, including the compatibility fixes needed for the current tabs APIs.
  • Adds a declarative GridModel scenario harness with 22 passing scenarios and 2 explicit todos after stabilization.
  • Completes Playwright component coverage with 16 passing tests and 3 explicit fixmes.
  • Adds 13 focused GridLayout showcase examples plus an interactive inspector for tracks, item areas, placeholders/stack state, and serialized JSON.
  • Stabilizes GridLayout behavior by enforcing splitter minimum sizes, correcting fractional and multi-track resize math, normalizing irregular removal with multiple unused lines, enforcing non-resizable split/drop rules, and isolating nested GridLayout drag contexts.

Validation

  • npx vitest run packages/grid-layout/test/GridModel.scenarios.test.ts --reporter=dot — 22 passed, 2 todo
  • PLAYWRIGHT_CHROMIUM_EXECUTABLE_PATH='/Applications/Google Chrome.app/Contents/MacOS/Google Chrome' npx playwright test --config=playwright-ct.config.ts packages/grid-layout/src/__tests__/__component__/GridLayout.playwright.test.tsx --project=chromium --reporter=line — 16 passed, 3 fixme
  • Targeted npx biome check --diagnostic-level=error ... — passed
  • npm run build --workspace @heswell/grid-layout — passed
  • npm run build:worker and npm run typecheck — passed
  • npm run showcase:prod — production bundle built and the GridLayout route served successfully

Remaining fixmes

  1. Palette template drop onto an empty placeholder
  2. Palette template centre replacement of existing content
  3. Tab reorder by dragging within a tabstrip

heswell and others added 12 commits August 23, 2026 16:08
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@heswell
heswell changed the base branch from heswell-copilot-grid-layout-tabs-iteration to main August 24, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant