feat(mobile): WIP redesign core app surfaces - #5373
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.


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.