Skip to content

sessions - hide diff stats from changes view button#304738

Merged
bpasero merged 1 commit intomainfrom
ben/rough-earthworm
Mar 25, 2026
Merged

sessions - hide diff stats from changes view button#304738
bpasero merged 1 commit intomainfrom
ben/rough-earthworm

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Mar 25, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 25, 2026 12:37
@bpasero bpasero enabled auto-merge (squash) March 25, 2026 12:37
@bpasero bpasero self-assigned this Mar 25, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.114.0 milestone Mar 25, 2026
Copy link
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

This PR adjusts the Agent Sessions “Changes” view toolbar to remove per-button diff statistics, leaving those actions icon-only while keeping overall line counts available in the view’s summary/header.

Changes:

  • Remove diff stat label rendering (added/removed counts) from the “View changes” related toolbar buttons.
  • Delete now-unused CSS rules that styled the removed diff stat label in toolbar buttons.

Reviewed changes

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

File Description
src/vs/sessions/contrib/changes/browser/media/changesView.css Removes toolbar-button-specific styling for diff stat labels that are no longer rendered.
src/vs/sessions/contrib/changes/browser/changesView.ts Stops generating a Markdown diff-stats label for view-related actions and hides their labels (icon-only).

@bpasero bpasero merged commit 54b102d into main Mar 25, 2026
39 of 45 checks passed
@bpasero bpasero deleted the ben/rough-earthworm branch March 25, 2026 13:36
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