fix(mobile): iOS 18 iPad sidebar and thread feed start below the header#32936

merged8
main
fix/issue-32884
opened4d agobyJJulius Marminge
View on GitHubTwo-way sync enabledPR last imported 1d ago

Problem

On iPadOS / iOS before 26 (no Liquid Glass, so the bars are solid), two lists start underneath their column's header. The native column lays content out under the bar, but these lists add no top inset there:

  • In the iPad sidebar, the newest threads sit behind the title bar and the Search field.
  • In the right-hand inspector pane showing a subagent, the top of the loaded page is behind the pane's header. The "Load earlier activity" button can't be tapped, and the first rows are hidden under the title.

The same happens to the top of an ordinary thread feed, to the empty-thread placeholder, and to the first message sent into an empty thread, on iPad and iPhone.

Change

  • Sidebar: ThreadNavigationSidebar now uses UIKit's automatic content inset (contentInsetAdjustmentBehavior="automatic", with indicator insets) on every iOS version. Before, it was only on when glass was supported. UIKit then clears the bar and its stacked Search field. On iOS 26 the props are the same as before. The WORKSPACE_LIST_SCROLL_INSETS constant only existed to turn this off on iOS < 26, so it is removed.
  • Feed: ThreadFeed no longer takes a contentTopInset prop. Its only caller always passed 0.
    • When the feed does not get the native contentInset (iOS < 26), its top inset is useContentTopInset(), the column's measured top safe area. That is the bar height where the column lays content under the bar, and 0 on Android.
    • On iOS 26, where the feed already gets a native contentInset of the bar height, it stays 0.
    • This value drives the existing list-header spacer, the empty and unavailable placeholders, the scroll-indicator inset and the first-send anchor offset.
    • The header-material scroll check adds the bar height only on the native-inset path, so the spacer does not count as a scroll.
  • The doc comment on useContentTopInset now says it applies to solid bars too.

Scope and approval

Fixes pingdotgg/t3code#32884 Fixes pingdotgg/t3code#22543

Triaged via the automated issue flow. #22543 reports the sidebar half on its own.

Verification

  • vp run check:changed passes on the head 08e1fb258a (format, lint, knip, typecheck, react compiler, tests beside the changed files).
  • This is a visual bug, so the proof is device captures and no new test. Before is main at ca796d5211 (N1: 80e1e4d14d). After is eb39186ba6, f76e21be6c (comment-only on top), or 4980e33628, as each row says; the head 08e1fb258a only merges main with no apps/mobile changes. Both sides share the native fingerprint and Legend List 3.7.2. Main merged since eb39186ba6 changes only file preview, asset loading and the Android rail width (0 off Android) under apps/mobile. The changed inset paths are identical apart from comments.
  • Environment: pool iOS simulators driven by agent-device, 1x capture pixels per point, default text size, fresh dev servers seeded with synthetic Replay (dev) threads. iPad (7th generation) iOS 18 1080×810 landscape, iPhone Xs iOS 18 / 18.6 375×812 portrait, iPad Pro 11-inch (M4) iPadOS 26.0 1210×834 landscape. Each comparison is cropped to the pane it covers, or has the status strip cropped and generated titles and timestamps masked identically on both sides. Diff0 marks every changed pixel; DiffT uses 5% fuzz.
RowBeforeAfterDiff0DiffT (5%)
S1 iPad 7 iOS 18, light. Newest rows behind bar and Search → row 20 first below Search. ca796d5211 → eb39186ba6S1 beforeS1 afterS1 Diff0S1 DiffT 5%
S2 iPad 7 iOS 18, light, status strip cropped. Empty sidebar: "No threads yet" moves below Search. ca796d5211 → eb39186ba6S2 beforeS2 afterS2 Diff0S2 DiffT 5%
F1 iPad 7 iOS 18, light, detail pane crop. Parent thread top: first rows under the header → below it. ca796d5211 → eb39186ba6F1 beforeF1 afterF1 Diff0F1 DiffT 5%
F2 iPad 7 iOS 18, light, inspector crop. Subagent top: "Load earlier activity" under the header → 70 pt lower, header identical. ca796d5211 → eb39186ba6F2 beforeF2 afterF2 Diff0F2 DiffT 5%
F3 iPhone Xs iOS 18.6, light, status strip cropped. Full-screen subagent: control under the bar → just below it. ca796d5211 → 4980e33628F3 beforeF3 afterF3 Diff0F3 DiffT 5%
F4 iPad 7 iOS 18, light, detail pane crop. Empty thread: "No conversation yet" centred below the bar. ca796d5211 → eb39186ba6F4 beforeF4 afterF4 Diff0F4 DiffT 5%
F5 iPad 7 iOS 18, light, inspector crop. "Messages not cached" centred below the header, 35 pt lower. ca796d5211 → f76e21be6cF5 beforeF5 afterF5 Diff0F5 DiffT 5%
F8 iPhone Xs iOS 18, light, timestamps masked. Short settled feed: zero changed pixels. ca796d5211 → eb39186ba6F8 beforeF8 afterF8 Diff0F8 DiffT 5%
I2 iPad 7 iOS 18, light, inspector crop. Earlier-page error: control unreachable → error wraps below it. ca796d5211 → f76e21be6cI2 beforeI2 afterI2 Diff0I2 DiffT 5%
I3 iPhone Xs iOS 18, light, keyboard up, status strip cropped. First send into an empty thread: bubble under the bar → 16 pt below it. ca796d5211 → f76e21be6cI3 beforeI3 afterI3 Diff0I3 DiffT 5%
N1a iPhone Xs iOS 18.6, light, keyboard up, title/timestamps masked. New task → first send: bubble hidden under the bar → below it. 80e1e4d14d → 4980e33628N1a beforeN1a afterN1a Diff0N1a DiffT 5%
N1b iPhone Xs iOS 18.6, light, keyboard down, same masks. Reply 88 pt lower, clear of the bar. 80e1e4d14d → 4980e33628N1b beforeN1b afterN1b Diff0N1b DiffT 5%
G1 iPad Pro 11 (M4) iPadOS 26.0, light, inspector crop. Glass path: pixel-identical. ca796d5211 → 4980e33628G1 beforeG1 afterG1 Diff0G1 DiffT 5%

Recordings:

Evidence write-ups:

Seed and driver scripts: seed-rpc.mjs, driver.md, capture.py, seed.py, recapture-seed.mjs, repair-comparisons.py, repair-four-comparisons.py.

What else changed in the captures: only the moved content above. Inside each crop or mask, the following still differ:

  • the status-bar clock;
  • generated thread titles, Replay timestamps and durations;
  • Expo dev-client overlays.

The recordings also carry touch markers and grey boxes over model labels. In the I1 before recording the inspector is 266 pt wide; after, it is 260 pt. Run 3's Search recordings show a Working counter only on the after side.

Not checked:

  • Android. useContentTopInset returns 0 there, and the sidebar change is in the iOS-only branch.
  • The compact iOS 26 send lifecycle. The feed keeps its native-inset path there with unchanged values, and G1 shows the glass inspector pixel-identical.
Verification log
  • Main merged into the branch again (4980e33628, then 08e1fb258a), with no conflicts.
  • Reviews: three earlier rounds and a round after each later merge, each with two independent reviewers. No code defects found. Findings were missing or stale captures and body text, now addressed. On 4980e33628 both reviewers returned MERGE with no findings.
  • Proof audit: the matrix was re-derived on 4980e33628. Its two passes found clock, timestamp and parent-pane noise in S2, F2, F3, F5, F8, I2, I3 and N1, and a before/before G1 pair. Those comparisons were rebuilt as above. The auditor rebuilt all ten independently with zero pixel delta and returned sufficient.
  • A message-search jump with the keyboard up can land the target partly under the bar on iOS 18. It lands in the same place before and after this change (scrollToIndex with viewPosition: 0.2), so it is out of scope here.

AI-assisted; verified with the tests and tools listed above.

Checks

20 successful · 22 skipped
All checks passed
08e1fb2

Added 2 commits

  • Merge remote-tracking branch 'origin/main' into fix/issue-3288429cca6d
  • fix(mobile): iOS 18 sidebar and thread feed start below the headerdb1c4e6
Synced from GitHub

Approvability

Verdict: Approved at f76e21b

Macroscope's review found this PR approvable — This corrects sidebar and thread-feed spacing beneath iOS headers and aligns header appearance with scrolling. The changes are limited to layout behavior, with no persistent-data or external-system effects.

No code changes detected at 08e1fb2. Prior analysis still applies.

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

[GPT-6.1 Sol][Codex] on behalf of shivam.

Handoff from the issue-watch investigation of https://github.com/pingdotgg/t3code/issues/32884. I did not reproduce this on a device: the iOS/Android pools were empty and shiv was unreachable during my run. No app build, native interaction, screenshot or recording was produced, and no format/lint/typecheck or repository test suite ran.

Source investigation at 22dc6f9e98cc826ac3581aeeb0fb1852f49a43f1 found the sidebar's glass-gated automatic inset and the feed's pre-26 combination of automatic insets disabled plus contentTopInset={0}. The sidebar overlaps the existing work in https://github.com/pingdotgg/t3code/pull/22550. I also identified the separate end-follow/snapback path covered by https://github.com/pingdotgg/t3code/pull/31907; I did not change that patch.

A local alternative fix, f253083cc3629d72cf6ca12b4dd97e9a48d51df2, enabled automatic insets for the sidebar and all iOS thread routes. Independent source review found no definite new defect, but blocked readiness on missing native evidence and repository checks. The only executed validation was git diff --check, which passed. That local commit was never pushed and is being left behind now that this PR owns the issue. No runtime evidence from my work is available to add to yours.

Synced from GitHub

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

ThreadFeed now gets its top inset from useContentTopInset and adjusts header-material visibility using barInset when native automatic insets are enabled. The thread navigation sidebar now uses UIKit automatic inset adjustment on iOS.

Changes: iOS Scroll Insets

Layer / File(s)Summary
ThreadFeed top-inset handling
apps/mobile/src/components/ui/useBarInset.ts, apps/mobile/src/features/threads/ThreadFeed.tsx, apps/mobile/src/features/threads/ThreadDetailScreen.tsx
The inset documentation now describes the iOS bar height for glass or solid bars and manual insets when UIKit automatic insets are disabled. ThreadFeed removes the contentTopInset prop, reads the column top inset from useContentTopInset, and uses barInset for the header-material visibility offset when native automatic insets are enabled.
Sidebar automatic inset adjustment
apps/mobile/src/features/layout/workspace/workspace-list-insets.ts, apps/mobile/src/features/threads/ThreadNavigationSidebar.tsx
The shared WORKSPACE_LIST_SCROLL_INSETS configuration is removed. On iOS, the sidebar list now enables automatic scroll-indicator and content-inset adjustment.

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

Change: Bug fix · Severity of issue fixed: Medium

Possibly related issues

  • pingdotgg/t3code#31867: The ThreadFeed inset change affects the position of “Load earlier activity.” The PR does not change the reported end-follow behavior.
  • pingdotgg/t3code#22543: The sidebar now enables automatic content-inset adjustment on iOS, addressing the reported content positioning under the header and Search field.
Merge Risk: External image Minimal · up to f76e2

This change makes the iPad sidebar and thread feed start below their headers on iOS 18. No concrete merge-blocking risk was found in the supplied changes.

Pre-merge checks: Passed 3 · Failed 1

Failed Failed checks: 1 warning

Check nameStatusExplanationResolution
Description checkWarningThe description clearly explains the problem, change, and detailed verification. The Scope and approval section links triaged issues but does not include a maintainer approval comment or explain why t…Add a link to the triaged issue or discussion with the maintainer’s explicit approval of the direction and scope, including the approval comment. If the change qualifies as a small, focused fix of an obvious bug without prior approval, expl…
Passed Passed checks: 3 passed

Check nameStatusExplanation
Title checkPassedThe title clearly identifies the iOS 18 sidebar and thread-feed inset fix.
Linked Issues checkPassedOpen, directly linked issue #32884 requires sidebar content to remain visible below its header. ThreadNavigationSidebar now enables UIKit automatic content and scroll-indicator insets on iOS. `Threa…
Out of Scope Changes checkPassedAll reviewed changes support issue #32884. The sidebar inset change fixes the left list. The feed inset change and removal of its contentTopInset override fix the subagent pane and apply the same in…
Full details: Linked Issues check

Explanation

Open, directly linked issue #32884 requires sidebar content to remain visible below its header. ThreadNavigationSidebar now enables UIKit automatic content and scroll-indicator insets on iOS. ThreadFeed uses useContentTopInset() when UIKit does not apply automatic insets, and keeps the manual inset at zero on the automatic path. This clears the left sidebar and subagent feed below their bars. The PR description reports iPadOS 18 captures for both cases, including the earlier-activity button below the inspector header. Issue #32884 does not require an automated test, so the absence of a new test is not an unmet requirement.

Full details: Out of Scope Changes check

Explanation

All reviewed changes support issue #32884. The sidebar inset change fixes the left list. The feed inset change and removal of its contentTopInset override fix the subagent pane and apply the same inset to the list spacer, placeholder, indicators, and anchors. Removing WORKSPACE_LIST_SCROLL_INSETS follows from enabling automatic insets on the sidebar. The useContentTopInset comment documents the new solid-bar use. No unrelated change is evident.

Full details: Description check

Explanation

The description clearly explains the problem, change, and detailed verification. The Scope and approval section links triaged issues but does not include a maintainer approval comment or explain why the change qualifies for the small, obvious bug exemption.

Resolution

Add a link to the triaged issue or discussion with the maintainer’s explicit approval of the direction and scope, including the approval comment. If the change qualifies as a small, focused fix of an obvious bug without prior approval, explain why.

Finishing Touches

Generate docstrings

  • Commit to this branch
  • Create a new PR
Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
External image Recent review info

External image Run configuration

External image Commits

Reviewing files that changed from the base of the PR and between eb39186ba693087529323f5e4da65018d8377ad5 and f76e21be6cdcf0936a28521badaf66e4a85bb9cc.

External image Files selected for processing: 2

  • apps/mobile/src/components/ui/useBarInset.ts
  • apps/mobile/src/features/threads/ThreadFeed.tsx
External image Files skipped from review as they are similar to previous changes: 2

  • apps/mobile/src/components/ui/useBarInset.ts
  • apps/mobile/src/features/threads/ThreadFeed.tsx

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

Added 1 commit

  • Merge remote-tracking branch 'origin/main' into fix/issue-32884eb39186

Added 1 commit

  • fix(mobile): inset comments describe the manual contentInset pathf76e21b

Added 4 commits

  • Merge remote-tracking branch 'origin/main' into pr-3293622c0b10
  • Merge remote-tracking branch 'origin/main' into pr-329365d99419
  • Merge remote-tracking branch 'origin/main' into pr-329364980e33
  • Merge remote-tracking branch 'origin/main' into pr-3293608e1fb2