Skip to content

plan widget fixes#312631

Merged
justschen merged 5 commits intomainfrom
justin/carbink
Apr 27, 2026
Merged

plan widget fixes#312631
justschen merged 5 commits intomainfrom
justin/carbink

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

some improvements to language and rendering

Copilot AI review requested due to automatic review settings April 26, 2026 20:26
@justschen justschen enabled auto-merge (squash) April 26, 2026 20:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Improves the plan review widget’s button layout and refines the language shown for “exit plan mode” actions, making the UI clearer and better aligned with intended workflows.

Changes:

  • Adjusts plan review footer button ordering so Approve/Reject stay grouped and “Provide Feedback” can be right-aligned.
  • Adds CSS to right-align the “Provide Feedback” button within the plan review footer layout.
  • Updates Copilot CLI “exit plan mode” action labels/descriptions to clearer wording.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/widget/chatContentParts/media/chatPlanReview.css Adds a flex-alignment rule to push the “Provide Feedback” button to the right in the footer.
src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatPlanReviewPart.ts Reorders Reject vs Feedback button creation and adds a class used for CSS-based alignment.
extensions/copilot/src/extension/chatSessions/copilotcli/node/exitPlanModeHandler.ts Updates action labels/descriptions for plan-exit choices presented to the user.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 2

…exitPlanModeHandler.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
eleanorjboyd
eleanorjboyd previously approved these changes Apr 26, 2026
@justschen justschen closed this Apr 26, 2026
auto-merge was automatically disabled April 26, 2026 21:05

Pull request was closed

@justschen justschen reopened this Apr 26, 2026
@justschen justschen merged commit f7696ee into main Apr 27, 2026
26 checks passed
@justschen justschen deleted the justin/carbink branch April 27, 2026 01:29
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants