Repository navigation
fix(mobile): keep iPad tab bar within narrow sidebars - #26198
Conversation
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — The change constrains the iPad sidebar tab overlay to fit narrow sidebars and the available window width. Its effect is limited to layout, with non-iPad sizing and navigation behavior unchanged. You can add or adjust custom eligibility rules. Learn more. |
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Sidebar overlay width calculation apps/mobile/src/features/workspace/WorkspaceNativeTabs.ios.tsx |
On iPad, the overlay width uses the minimum of 280 points, the preferred list-pane width or window width, and window width. Other devices retain the previous rule. |
Priority:
Estimated code review effort: 2 (Simple) | ~5 minutes
Change: Bug fix
Possibly related PRs
- pingdotgg/t3code#24182: Changes iPad native workspace sizing and sidebar behavior during resize.
- pingdotgg/t3code#24206: Changes the iOS workspace tab overlay in
WorkspaceNativeTabs. - pingdotgg/t3code#24666: Changes the iPad sidebar tab bar layout in the native primary screen.
Suggested reviewers: pixpmusic
Merge Risk: 
Minimal · up to 886a8
-
The sidebar width adjustment is mergeable based on the reviewed evidence.
Pre-merge checks: 
4
-
Full details: Description check
-
Explanation
The description explains the problem, implementation, affected environment, verification steps, test results, and provides before-and-after screenshots. It does not include a dedicated Scope and approval section, but the remaining required information is substantially complete for this focused bug fix.
-
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
Comment @coderabbitai help to get the list of available commands.
Narrowing the iPad sidebar leaves the floating tab bar at its preferred width, so it extends into the chat composer. Cap the iPad bar at 280 points, the native sidebar's minimum width, and constrain it to the available width. This also tightens the spacing between the icons.
Verified on a 13-inch iPad Air simulator (iOS 26.5): reproduced the overlap by dragging the sidebar narrower, then confirmed the fix in landscape and portrait and switched between Chats, Usage, and Settings. iOS 27 was not verified.
vp run check:changedpasses; React Doctor reports 100/100. This is a visual layout fix, so the screenshots are the regression evidence.Implemented and verified with gpt-6-astra through the Codex harness in T3 Code.