Skip to content

Center the API overview page - #816

Merged
snario merged 1 commit into
mainfrom
agent/center-api-overview
Aug 15, 2026
Merged

Center the API overview page#816
snario merged 1 commit into
mainfrom
agent/center-api-overview

Conversation

@snario

@snario snario commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Center the OpenAPI overview/landing route, including the authored Tempo API overview at /docs/api.
  • Keep generated endpoint group pages full width for their two-column request/response playgrounds.
  • Extend the wide-viewport regression test to cover the overview, authored guide, and endpoint-reference layouts.

Root cause

The earlier fix updated OpenApiGuide, which covers authored pages beneath the OpenAPI mount, but /docs/api is rendered through the separate OpenApiPage landing branch. That branch still explicitly requested content.width = "full", collapsing the centering gutter even though the overview has no right-hand operation panel.

Impact

The Tempo API overview now uses the same centered readable column as other single-column docs. Generated endpoint reference groups are unchanged.

Screenshots

Before

Before: Tempo API overview left-aligned beside the sidebar

After

After: Tempo API overview centered in the standard readable column

Validation

  • pnpm build
  • pnpm exec biome check e2e/openapi-layout.test.ts
  • CI=true pnpm exec playwright test e2e/openapi-layout.test.ts --project=chromium

@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tempo-docs Ready Ready Preview Aug 15, 2026 1:02am

Request Review

@snario
snario marked this pull request as ready for review August 15, 2026 01:06
@snario
snario merged commit 38d21c7 into main Aug 15, 2026
12 checks passed
@snario
snario deleted the agent/center-api-overview branch August 15, 2026 01:07
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