fix(mobile): Android Git overview scrolls to every linked pull request#33496

Problem

On Android, the Git overview (thread Git controls → More, or the routed Git sheet) can't be touch-scrolled. After dragging the sheet to full height by its header, slow drags over the action rows or the linked-PR rows and upward flings all leave the list where it is, so later linked pull requests can't be reached. Reproduced on main on a Pixel 9 emulator (Android 16, API 36) with a thread holding 12 linked PRs, in both presentations.

Change

Material's bottom sheet sends drags to the first nested-scrolling native view it finds, and three things put the wrong view first:

• GitOverviewSheet drew its AppBar, whose Compose buttons are nested-scrolling views, before its list. It now renders the content first and reverses the column on Android (android:flex-col-reverse), so the header still sits on top. This is the same layout #28272 used for GitCommitSheet, and the other Git sheets use it too.
• In the inspector sheet, the Files and Git panes are prewarmed together. The hidden Files pane mounted first, so the sheet locked onto its header. The Android inspector host now gives the visible pane the lower zIndex, which Fabric mounts first. Hidden panes keep opacity: 0, pointerEvents="none" and no-hide-descendants.
• On Android, a RefreshControl wraps the list in a layout that the sheet reads as unable to scroll up, so after scrolling down, the next downward drag collapsed the sheet instead of scrolling back. Android bottom sheets (the routed sheet, and the inspector when it's shown as a sheet) no longer attach it; #17177 made the same call before a later refactor brought it back. Pull-to-refresh never fired in those sheets on main (the sheet takes the drag; see D1 below), so no working gesture is lost. The routed sheet's AppBar ⋮ → Refresh repository status still refreshes; the inspector sheet relies on the background status refresh, as it effectively did before. The Android tablet inspector column and iOS keep pull-to-refresh.

iOS layout is unchanged: there, AppBar only sets native header options and draws nothing in the column.

Scope and approval

Fixes pingdotgg/t3code#33445

Triaged via the automated issue flow. Open PR #33223 makes the same one-line inspector-host zIndex change for the Files sheet; the two lines are identical.

Verification

• vp run check:changed on cf0d8387fd: format, lint, knip, typecheck, React compiler, nearby tests (7 passed).
• No automated test: the defect is in how Material's native bottom sheet picks its scroll child, which the Node and browser harnesses can't reach. Proof is device recordings.
• Review round on b5b8f16b5d with two independent reviewers: no code findings. One reworded comment followed (cf0d8387fd, comment only).

Evidence

Before = main ee59ab22a8 (merge-base), after = d532013f5b. Both ran on the same current native dev clients (Android 3445feaf, iOS edb0efb7) with a fresh Metro bundle and a fresh database per revision, seeded by the same RPC driver: a synthetic repository, 60 files, 12 linked public PRs. The final head adds merges of main after d532013f5b. They leave the inspector host, inspector panes, the Git overview, the sheet wrappers, ScreenContent and Stack.tsx byte-identical, but they do change the surrounding mobile layout (AdaptiveWorkspaceLayout, the workspace pane context, NativeAppBar.android and AndroidScreenHeader) and the unrelated t3-highlights native module. The captures below were not rerun on the final head.

Environment: Pixel 9 emulator (Android 16, 1080×2424, font scale 0.94); Pixel Tablet emulator (landscape 2560×1600, 2×); iPhone 17 Pro simulator (iOS 27.2, 1206×2622, 3×). Full surfaces, no crops.

Scripted sequence for A1/A2, the same on both sides: expand the sheet by its header; A slow drag up over the actions; B slow 450 px drag down; C slow drag up over the PR rows; D fling up; E tap the last linked PR. Timestamps per step are in the full reports linked below.

Settled initial frames, before → after (Diff0 = any RGB change, DiffT = change above 10%):

¹ iOS stills mask the status band (the same 1206×180 white rectangle on both sides) because the simulator's status override re-animates the battery; unmasked originals are in the run 4 archive.

Expanded → D motion masks (within each run): inspector before 0 / 0 px, after 430,590 / 248,142 px; routed before 0 / 0 px, after 409,165 / 234,972 px.

What else changed: nothing visible beyond the scrolling. Every initial frame in the table is identical at zero threshold except two one-level antialiasing residues that persisted across retakes (A2: 6 px on the gesture bar; A5: 11 px along the disabled “Push” label), both 0 above 10%. Android bottom sheets no longer attach RefreshControl, which never fired there (D1).

Not verified:

• The Android inspector sheet's ⋮ menu opens at the top of the screen, and tapping Refresh dismisses the sheet without refreshing (capture). It is an existing bug, the same on main, and not touched here.
• Hidden panes now stack above the visible one. The accessibility tree shows only the visible pane's nodes, and taps reach it (AX dump). TalkBack announcements, stylus and hover were not run.
• Android Files-sheet list scrolling: it doesn't scroll before or after here. That is a separate bug, addressed by open #33223.
• The thread preview with a populated conversation (the fixture thread is empty), and the device preview (no device target can be advertised on the test setup).
• Not captured: iPad inspector column, width transitions (fold/unfold), and data or permission variants (loading, non-repo, behind upstream, read-only) whose layout this diff doesn't change.
• A physical device on Android 17 (the reporter's).

Full reports with per-step timestamps, every capture, and the seed and driver scripts: run 2, run 4 (audit gaps), run 3 (tablet column), seed.mjs, capture-driver.py, make-diffs.py.

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