Skip to content

Polish multi-file diffs in Agents window#327442

Draft
romalpani wants to merge 1 commit into
microsoft:mainfrom
romalpani:agents/ui-improvements-multi-file-diffs
Draft

Polish multi-file diffs in Agents window#327442
romalpani wants to merge 1 commit into
microsoft:mainfrom
romalpani:agents/ui-improvements-multi-file-diffs

Conversation

@romalpani

@romalpani romalpani commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Present each changed file as an inset, rounded card in the Agents Changes view.
  • Show secondary file actions on hover or focus and align the collapse control with toolbar actions.
  • Highlight only the focused card with subtle, theme-aware active and keyboard-focus states.
  • Expand collapsed cards before running full-file expand or collapse actions.
  • Add bottom scroll clearance so the final card is not obscured by the horizontal scrollbar.

Before:
CleanShot 2026-07-24 at 21 19 56@2x

CleanShot 2026-07-24 at 21 20 12@2x

After:
CleanShot 2026-07-24 at 21 21 27@2x
CleanShot 2026-07-24 at 21 24 53@2x

CleanShot.2026-07-24.at.20.50.20.mp4

Validation

  • Core transpilation and extension build pass.
  • Changes action tests pass (7 tests).
  • Precommit hygiene and diff checks pass.
  • Full typecheck remains blocked by unrelated existing Copilot SDK and Agent Host errors.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Polishes multi-file diff cards in the Agents Changes view and improves collapsed-file actions and scrolling.

Changes:

  • Adds inset card styling, contextual actions, and focus states.
  • Expands collapsed cards before full-file actions.
  • Adds configurable bottom scroll clearance.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
sessionChangesEditor.ts Configures bottom clearance.
multiFileDiffEditor.css Styles cards, actions, and focus states.
changesActions.ts Expands cards before file-wide actions.
multiDiffEditorWidgetImpl.ts Includes bottom padding in scroll height.
multiDiffEditorWidget.ts Exposes bottom-padding configuration.

Comment on lines +137 to +140
.hc-black .agent-sessions-workbench .part.editor .multiDiffEntry:focus-within .header .header-content,
.hc-black .agent-sessions-workbench .part.editor .multiDiffEntry:focus-within .editorParent,
.hc-light .agent-sessions-workbench .part.editor .multiDiffEntry:focus-within .header .header-content,
.hc-light .agent-sessions-workbench .part.editor .multiDiffEntry:focus-within .editorParent {
display: flex;
align-items: center;
justify-content: center;
padding: 3px;
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.

3 participants