Repository navigation
Center the conversation and fix Replies and drawer spacing - #1112
Merged
TheGreatAxios merged 10 commits intoOct 9, 2026
Merged
Conversation
pratikbuilds
marked this pull request as ready for review
October 7, 2026 15:35
TheGreatAxios
approved these changes
Oct 8, 2026
TheGreatAxios
left a comment
Contributor
There was a problem hiding this comment.
This is perfect -- aligning even better with the original mockups and spacing targets. LGTM
pratikbuilds
force-pushed
the
cl-1109-centered-conversation
branch
from
October 9, 2026 10:56
9553710 to
f430e74
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The conversation, day divider, and input share a centered column capped at 1200px when side panels are closed. Opening Replies, the bench drawer, or both keeps the input 16px from the nearest panel. User bubbles retain their 16px inset. The desktop drawer is a flat side panel with one divider, avoiding a nested card surface.
Fixes #1109.
Evidence
Conversation alignment — the timeline, divider, and input now share one centered column. Before is the original issue screenshot; after is the current local preview.
Replies spacing — the large empty gap beside Replies is replaced by a 16px input inset. User bubbles keep their inset. These close-ups show the reported gap and the final layout; full after screenshot.
Details drawer — the input gap drops from 81.5px to 16px. The nested rounded card becomes a flat panel with one continuous divider. Both screenshots use the same conversation and 2048px viewport.
After:
bun test e2e/browser/zero-to-one.test.tspasses against a disposable Postgres database: 1 test, 466 assertions. Covers 390–2048px widths, 860/861px phone boundary, drawer and Replies toggles, both panels open, 16px/20px reserved scrollbar space, scrolling, prose width, and overflow. Mocked worker reply, follow-up, artifact creation, routine brief, and rename persistence pass; browser errors are empty.bun run build, typecheck, lint, formatting, andgit diff --checkpass.bun run checkstill fails on two existing packages withbun testscripts and no test files:@corbits/url-pathand@corbits/workflows. DB-backed suites were checked separately with the browser command above; the root check ran withoutDATABASE_URL.The merge resolution keeps main’s deletion of the standalone worker-reply suite and moves this PR’s layout checks into the consolidated zero-to-one flow. The CSS and design changes are unchanged.
Merge Danger
Door: two-way
CSS layout changes; no schema or runtime changes.
Blast Radius: layout
Conversation, input, side panel spacing, and bench drawer appearance. The phone drawer remains a full-width overlay.