fix(app): restore session timeline width cap#483
Conversation
|
Warning Rate limit exceeded
To continue reviewing without waiting, purchase usage credits in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Restore the session timeline medium-width cap to the explicit
800pxvalue from #464, and add a right-panel-hidden regression assertion at 1280px.Why
Fixes #479.
#464 fixed #456 by changing the session timeline from
md:max-w-200tomd:max-w-[800px], because PawWork's 13px root font makesmax-w-200resolve to about 650px. The composer remains capped at 720px in the same viewport band, so the old timeline cap makes the composer visibly wider than the message flow.The regression was reintroduced during the #440 UI slice merge chain: #461 first changed the two timeline caps back to
md:max-w-200, and #460 later kept that same file state while removing dead delete-session code frommessage-timeline.tsx.Related Issue
Closes #479
Human Review Status
Pending. A human should make the final merge decision after reviewing the final diff and verification evidence.
Review Focus
Check that this only restores the lost timeline width cap and adds one focused right-panel-hidden coverage point. It does not change composer sizing, right-panel behavior, or broader layout tokens.
Risk Notes
Low. This is a two-line session layout restore plus one E2E coverage point. It does not touch persistence, packaging, permissions, dependencies, or platform code.
How To Verify
Screenshots or Recordings
Not attached. The regression E2E now explicitly asserts
#right-panel[aria-hidden="true"]at 1280px and verifies the timeline/composer bounding-box contract.Checklist
dev, and my PR title and commit messages use Conventional Commits in English