Skip to content

Fix run detail page gutters - #2331

Open
federicodeponte wants to merge 1 commit into
mainfrom
codex/run-detail-margins-20260727
Open

Fix run detail page gutters#2331
federicodeponte wants to merge 1 commit into
mainfrom
codex/run-detail-margins-20260727

Conversation

@federicodeponte

Copy link
Copy Markdown
Member

Closes #2330.

What changed

  • Treat only the dedicated /runs/[id] route as a standard page.
  • Preserve full-bleed behavior for the /runs collection.
  • Add DOM regression coverage for detail, collection, trailing-slash, and standalone share routes.

Root cause

The shell prefix-matched /runs for all descendants, so /runs/[id] inherited the collection shell: no gutters and overflow-hidden.

Verification

  • Focused regression: 4/4 passed.
  • Full web suite: 1,023 passed, 1 skipped.
  • Web lint: passed with existing warnings.
  • Web production build: passed.

@federicodeponte
federicodeponte marked this pull request as ready for review July 27, 2026 21:24
@federicodeponte

Copy link
Copy Markdown
Member Author

The required Web lint status is failing in the repository-wide production dependency audit before PR-specific validation. This is reproduced from the unchanged main dependency graph and tracked in #2332. All PR-specific tests, full web tests (1,023 passed), lint, and production build pass locally.

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.

Fix missing gutters on dedicated run detail pages

1 participant