Repository navigation
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 59 minutes. View limit detailsLimit details: You’ve used the included review currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (4)
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 |
|
Review fixes pushed in 64f0e7f. The footer-selector finding is addressed and its thread has a reply/resolution. Full The updated changes are ready for another review; the latest review checks are still running. |
Bug
The sidebar's viewport-oriented sizing and non-shrinkable flex children can put composer actions outside the native host. At narrow widths, the secondary reasoning-effort label also competes with the remaining actions.
Fix
Use host-relative root/body height, keep flex children shrinkable, constrain the utility/footer areas, and hide only the secondary effort label in narrow composer containers. The model/effort remains available through the trigger's accessible name and menu. This stylesheet does not shrink fonts or disable zoom; the font cascade fix is separate.
Validation
pnpm testpasses on this independent branch; assertions extend the existing bootstrap test rather than adding a layout suite.The full responsive smoke uses the refreshed components plus the separate font fix; it is not an authenticated conversation test or a claim that the older vendor UI has identical geometry. This PR's source is otherwise independent of those changes. The native companion is unchanged.
Generated by GPT 6.1 Sol in Codex, using T3 Code.

Confidence Score: 5/5
No outstanding findings block merging. The footer selector now reaches the shipped component.Summary
The PR adds sidebar sizing rules and hides the secondary effort label in narrow composer containers.
Reviews (2) · Last reviewed commit: "Apply host sizing to the shipped compose..." · Reviewed by Greptile