Fix for issue 123 - #6066
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 |
ApprovabilityVerdict: Approved 4d7886f Straightforward UI enhancement adding a 'Hide Terminal' button using existing state management. The change is purely additive with an optional prop, backwards-compatible, and doesn't alter existing behavior. You can customize Macroscope's approvability policy. Learn more. |
Fixes issue #123
Note
Low Risk
UI-only visibility toggle reusing existing
setTerminalOpenstate; no changes to terminal lifecycle or server behavior beyond hiding the drawer.Overview
Adds a Hide Terminal control to the thread terminal UI so users can collapse the drawer without closing terminal sessions.
ChatViewwires a newonHideDrawerhandler that setsterminalOpentofalsevia the terminal UI store for the current thread.ThreadTerminalDraweraccepts optionalonHideDrawerand renders a chevron-down action in both the panel and compact toolbars (next to new/split/close), only when that callback is provided.Reviewed by Cursor Bugbot for commit 4d7886f. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add 'Hide Terminal' button to
ThreadTerminalDrawerAdds an optional
onHideDrawerprop toThreadTerminalDrawer. When provided, a 'Hide Terminal' button (ChevronDown icon) appears in both drawer and panel action bars.PersistentThreadTerminalDrawersupplies the handler, which setsterminalOpen=falsefor the thread via the terminal UI state store.Macroscope summarized 4d7886f.