Repository navigation
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| const requestChrome = props.activePendingUserInput ? 430 : 270; | ||
| const composerOverlapHeight = activePendingRequest | ||
| ? requestChrome + Math.max(insets.bottom, 12) | ||
| : composerChrome + composerBottomInset; |
There was a problem hiding this comment.
Fixed pending card inset height
Medium Severity
Feed bottom inset for pending approval or user-input overlays uses hardcoded heights (270 and 430) instead of measuring the redesigned cards. Tall approval details or multi-option input steps can exceed those constants and leave messages hidden under the overlay.
Reviewed by Cursor Bugbot for commit 62b3883. Configure here.
ApprovabilityVerdict: Needs human review This is a major mobile UI redesign introducing new components (ModelPickerSheet, thread header status), new features (model favorites), and significant changes across core app surfaces. The WIP designation and an unresolved bug report about hardcoded overlay heights warrant human review. You can customize Macroscope's approvability policy. Learn more. |
62b3883 to
73cc66f
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
There are 3 total unresolved issues (including 2 from previous reviews).
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 73cc66f. Configure here.
|
Note 🤖 GPT-5.6 Sol responding on behalf of Theo We're closing this PR as we clean up the T3 Code backlog. Thank you for taking the time to put this together. This is an explicit work-in-progress redesign of the home header, composer controls, model sheet, thread list, and related mobile components. The 27-file visual change has no selected final design that can be reviewed as one finished behavior. If you believe we closed this in error, please reopen the PR and leave a comment explaining what we missed. |


Warning
WIP: This redesign is still under active visual verification.
What Changed
Why
The previous React Native surfaces were too small, crowded important titles with utilities, and produced an unreliable composer when the keyboard opened. This update brings the mobile client closer to the rebuilt native direction while keeping the implementation shared across iOS and Android.
UI Changes
Captured from the repository's iPhone 6.9 iOS simulator workflow.
Thread list
Thread detail and composer
Simulator evidence: baseline run and final WIP run.
Verification
packages/contractserror already present on main:orchestration.tsreports amessageproperty onOption<unknown>.Checklist
mainBuilt with GPT-5.6 Codex in the T3 Code harness.