[PixPMusic]

Problem

Android chat and new-thread panels have square top corners below the header. The screen-host refactor in #23467 moved the header inside the rounded container, so clipping happens above the header instead of around the content.

Change

Place the Android rounded content pane below the header while retaining the shared screen host. Let fallback screens supply their header separately, including the draft shown when no model is selected. The iOS header/body order and background remain the same.

Scope and approval

Small, focused fix for an obvious visual regression: restore the existing 28 dp Android content corners. Based directly on main at 807319c276c6831b2c219640e9da5b38019a8f1d, independent of the toast changes in #23968.

Verification

• vp run check:changed: format, lint, knip, and affected typechecks pass.
• Native Pixel Fold emulator, folded and unfolded: existing chat content and new-thread drafts have rounded corners below the header; navigation between drafts and chats preserves the composer and loaded conversation.
• All captures use demo fixtures and the Moonbase Terminal environment label.
• Not exercised: native iOS, unavailable-thread/loading states, and the disconnected/no-model fallback.

Layout	Before	After
Folded

Unfolded chat · Folded draft · Unfolded draft

Model: GPT-6.1 Sol | Harness: Codex in T3 Code

[macroscopeapp]

Approvability

Verdict: Approved at d054868

Macroscope's review found this PR approvable — This is a focused Android UI regression fix that restores rounded thread-content corners while preserving header behavior and leaving iOS unchanged. The two-file change is localized and introduces no new capability, schema, or sensitive behavior.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

Warning

Review limit reached

Only developers with an assigned seat can use this organization's usage-based review budget, and seats here are assigned manually. Ask an admin to assign a seat, or change the review continuation mode in Billing.

Next included review available in 21 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 220eb73
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 4b538067-f817-4bcb-88b1-0d5c977d13fb

Commits

Reviewing files that changed from the base of the PR and between 220eb73 and 925c5e1.

Files selected for processing: 2

• apps/mobile/src/features/threads/NewTaskDraftScreen.tsx
• apps/mobile/src/features/threads/ThreadScreenHost.tsx

No actionable comments were generated in the recent review.

ℹ️ Recent review info

⚙️ Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: 807319c
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: b0248f44-b562-498c-9f40-c71c3bf6750c

📥 Commits

Reviewing files that changed from the base of the PR and between 807319c and d054868.

📒 Files selected for processing: 2

• apps/mobile/src/features/threads/NewTaskDraftScreen.tsx
• apps/mobile/src/features/threads/ThreadScreenHost.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 6 remain after this review.

Walkthrough

Fallback thread content can now provide a separate header. The outlet renders headers for both content types and applies a rounded, clipped body wrapper on Android. The “New thread” app bar now uses the fallback header prop.

Changes: Thread layout

Layer / File(s)	Summary
Fallback header and platform-specific body layout apps/mobile/src/features/threads/ThreadScreenHost.tsx, apps/mobile/src/features/threads/NewTaskDraftScreen.tsx	The fallback content accepts an optional header. The outlet renders the header and wraps the body in a rounded, clipped view on Android. The “New thread” app bar is passed through the header prop.

Priority: Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Possibly related issues

• pingdotgg/t3code#22737 — The issue concerns Android draft header layout, which these changes also affect.

Suggested labels: bugfix, size:S

Suggested reviewers: juliusmarminge

Merge Risk: Minimal · up to d0548

No merge-blocking risk is established for this change.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly and concisely describes the primary change: restoring rounded Android chat content corners.
Description check		The description covers the problem, change, scope and approval rationale, verification results, screenshots, untested states, and agent details. It follows the repository template and provides focused…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Description check

Explanation

The description covers the problem, change, scope and approval rationale, verification results, screenshots, untested states, and agent details. It follows the repository template and provides focused UI evidence.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}

[maria-rcks]

Note

Written by gpt-6.1-sol on behalf of Maria

thanks @PixPMusic. separating the header from the clipped content fixes the boundary without replacing the shared screen host or composer. current main still contains that regression.

i merged github/main at 220eb73524ca into your writable branch, preserving your commit and the original two-file fix unchanged. head: 925c5e1478dc195d82e503d982153f9da087b81f. no replacement, rebase, or force push.

verification:

• all 28 checks are green; git.cafe reports mergeable.
• blacksmith: vp run check:changed 220eb73524ca7882d53b6a12c824ccbcd97ad279 passed format, lint, knip, and affected typechecks. three existing navigation/draft suites passed, 23 tests total.
• two independent code reviews of this exact head found no issues, including loading, unavailable, branch-preparation, no-model fallback, and draft-to-live composer paths.
• i inspected your original folded before/after images at full size and preserved the evidence in the description. those are your original captures, not new captures of this head.

unverified: current-head native android clipping, keyboard behavior, and ios runtime. this vm has no attached android device, and t3 device access is disabled. chrome cannot verify native layout. before merging, confirm rounded content below the header in chat and no-model drafts, and that draft-to-live navigation keeps the composer and keyboard. coderabbit skipped this head because its review-seat limit was reached; its prior review had no findings.