Skip to content

fix(mobile): thread titles stay on one line instead of reflowing - #28853

Closed
mapringg wants to merge 1 commit into
pingdotgg:mainfrom
mapringg:fix/mobile-thread-title-single-line
Closed

mapringg wants to merge 1 commit into
pingdotgg:mainfrom
mapringg:fix/mobile-thread-title-single-line

Conversation

@mapringg

@mapringg mapringg commented Nov 1, 2026

Copy link
Copy Markdown

Problem

Thread titles can wrap or unwrap as the status label changes width, changing the list row height and shifting nearby rows. A long seeded title reproduces the two-line layout.

Change

Limit the shared thread-list row title to one line, truncating long titles. This also applies to the workspace sidebar that shares the row.

Scope and approval

Submitted as a small, focused fix of an obvious row-height instability, with no prior issue. It changes only the title line limit. This reverses the deliberate two-line choice in #25415 and may be declined; there is no prior maintainer approval for that reversal.

Verification

On an Android API 36 emulator in dark mode, the same long showcase title wraps before and truncates to one line after. Screenshots demonstrate the layout; I did not record a live status-width transition. iOS was not run.

node_modules/.bin/vp run check:changed passed at this branch head in the previous session; Sol review approved the diff. No fix code changed during evidence collection.

Before — thread title After
Before After

Code written by Claude Opus 5.5; evidence and PR filing by GPT-6.1 Sol, both in T3 Code.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Nov 1, 2026
@macroscopeapp

macroscopeapp Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Needs Human Approval

Macroscope's review found this PR requires human approval — Mobile thread titles will use one line instead of two, displaying less text in home and sidebar lists. This changes the default presentation and requires a human decision on that display tradeoff.

You can add or adjust custom eligibility rules. Learn more.

@coderabbitai

coderabbitai Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 1db31d0 and 7e5ed51.

Files selected for processing: 1
  • apps/mobile/src/features/threads/thread-list-v2-items.tsx

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

Walkthrough

The card-row thread title now displays at most one line instead of two.

Changes: Mobile thread title

Layer / File(s) Summary
Set title line limit
apps/mobile/src/features/threads/thread-list-v2-items.tsx
The card-row thread title’s numberOfLines limit changed from two to one.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Possibly related PRs

  • pingdotgg/t3code#25668: Changes the same mobile thread-row title layout, including title wrapping and alignment.
  • pingdotgg/t3code#26666: Measures title line height when titles can wrap to two lines, an assumption changed by this update.

Suggested labels: bugfix, size:XS, 📱 Native Change, vouch:unvouched

Suggested reviewers: juliusmarminge

Merge Risk: Minimal · up to 7e5ed

The one-line title change is ready to merge with no identified blocking risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title is concise, specific, and accurately describes the one-line thread title change.
Description check Passed The description covers the problem, change, scope rationale, verification results, limitations, screenshots, and agent attribution. It is complete for this focused UI fix.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Finishing Touches
Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@mapringg mapringg closed this Nov 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant