fix(mobile): working pill no longer bulges on its right end

[id0Sch]

Problem

On iOS with Liquid Glass, the floating "Working" pill has a bump on its right end when it's shown alone. The scroll-to-end arrow is hidden by fading its chevron and sliding it behind the capsule, but its GlassView keeps rendering. It ends up about 2px inside the capsule's right end cap, and the GlassContainer (spacing={12}) merges the two glass shapes into that bump.

Fix

• The arrow's glass style is "none" while hidden and "regular" when shown, with animate: true, so there is nothing to merge and the arrow still materializes next to the capsule when you scroll up.
• CONTROL_SEPARATION assumed a 16px gap, but gap-4 is 14px with the mobile 14px rem, which left the lone capsule about 1px off-center. It now uses the real gap.

The Android/non-glass fallback path is unchanged.

Verification

Typecheck, format and lint are clean for the changed file. Checked on an iPhone 17 Pro simulator (iOS 26.5) with a fresh dev-client build against an isolated backend: the lone pill, scrolling up to show the arrow, and tapping the arrow to collapse back.

Before (lone pill)	After (lone pill)

After, scrolled up, the arrow still separates as before:

Model: Claude Opus 5.5. Harness: Claude Code in T3 Code.

🤖 Generated with Claude Code

[macroscopeapp]

Approvability

Verdict: Approved at 3e60795

Macroscope's review found this PR approvable — This is a small, self-contained mobile UI fix that corrects the pill’s spacing and prevents the hidden scroll control’s glass material from creating a visual bulge. It does not introduce new capability, schema changes, or broader production behavior changes.

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

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 097a1709-cbb2-4440-b3db-169b13ed0444

📥 Commits

Reviewing files that changed from the base of the PR and between 3e6b450 and 18ec290.

📒 Files selected for processing (1)

• apps/mobile/src/features/threads/floating-working-control.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

Walkthrough

The floating working control now uses a 14-point row gap to calculate separation. Its native scroll-to-end glass effect changes between "regular" and "none" based on showScrollToEnd.

Changes

Floating working control

Layer / File(s)	Summary
Spacing and glass behavior apps/mobile/src/features/threads/floating-working-control.tsx	The row gap is 14 points, and CONTROL_SEPARATION derives from that value and CONTROL_HEIGHT. The native scroll-to-end glass effect uses an animated style that selects "regular" when showScrollToEnd is true and "none" otherwise.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to 18ec2

The hidden scroll-to-end arrow no longer adds native glass beside the working pill, and the separation calculation uses the 14-point mobile gap. The inspected caller and fallback behavior remain consistent; no merge-blocking risk is evident.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the mobile fix and the visible bug: the Working pill bulges at its right end.
Description check	✅ Passed	The description explains the problem, fix, and focused verification, and includes before-and-after screenshots. It omits the Scope and approval section and does not state whether an issue or maintaine…
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.

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

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

---

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