Replies: 2 comments
|
I keep running into this problem. When I want to quickly close the right panel, I instinctively click the X, which closes the entire app. |
0 replies
|
PR #4148 has been refreshed without opening a new PR: branch |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Before submitting
Area
apps/web
Problem or use case
The inline right panel can show Files, Diff, Terminal, Browser, and Plan surfaces, but its own header does not provide an explicit way to close the whole panel. Individual tabs expose an
×that closes that surface, while the whole-panel toggle lives outside the panel in the workspace titlebar.This makes dismissing the panel less discoverable: a user looking at the panel header can reasonably interpret the tab
×as the only close affordance, even though that action has different semantics.Proposed solution
Add a dedicated close button to the trailing edge of the inline right-panel header.
Why this matters
The close action becomes visible where users expect it, and the distinction between closing one surface and dismissing the entire panel becomes clear. This is especially useful when the panel was opened indirectly by selecting a file or diff from the chat timeline.
Smallest useful scope
Render one close control in the inline
RightPanelTabsheader and wire it to the existing panel-close callback. No settings, persistence, keyboard shortcut, or panel-state changes.Alternatives considered
rightPanel.togglekeybinding. These work, but they are spatially disconnected from the panel being dismissed.×. That would blur the distinction between closing a surface and closing the container.Risks or tradeoffs
Examples or references
The panel already has a whole-panel toggle in
PanelLayoutControls; this proposal makes the same action available in the panel header itself.Contribution
All reactions