From 54b102d97603fe6d9405a0796e45604dce63f76e Mon Sep 17 00:00:00 2001 From: Benjamin Pasero Date: Wed, 25 Mar 2026 14:36:19 +0100 Subject: [PATCH] sessions - hide diff stats from changes view button (#304738) --- .../contrib/changes/browser/changesView.ts | 8 +------- .../contrib/changes/browser/media/changesView.css | 15 --------------- 2 files changed, 1 insertion(+), 22 deletions(-) diff --git a/src/vs/sessions/contrib/changes/browser/changesView.ts b/src/vs/sessions/contrib/changes/browser/changesView.ts index 4c3c01bc186..3778ad29ecd 100644 --- a/src/vs/sessions/contrib/changes/browser/changesView.ts +++ b/src/vs/sessions/contrib/changes/browser/changesView.ts @@ -10,7 +10,6 @@ import { ICompressedTreeNode } from '../../../../base/browser/ui/tree/compressed import { ICompressibleTreeRenderer } from '../../../../base/browser/ui/tree/objectTree.js'; import { IObjectTreeElement, ITreeNode } from '../../../../base/browser/ui/tree/tree.js'; import { Codicon } from '../../../../base/common/codicons.js'; -import { MarkdownString } from '../../../../base/common/htmlContent.js'; import { Iterable } from '../../../../base/common/iterator.js'; import { Disposable, DisposableStore, MutableDisposable } from '../../../../base/common/lifecycle.js'; import { autorun, constObservable, derived, derivedOpts, IObservable, IObservableWithChange, ISettableObservable, ObservablePromise, observableSignalFromEvent, observableValue, runOnChange } from '../../../../base/common/observable.js'; @@ -659,7 +658,6 @@ export class ChangesViewPane extends ViewPane { this.renderDisposables.add(scopedInstantiationService); this.renderDisposables.add(autorun(reader => { - const { added, removed } = topLevelStats.read(reader); const outgoingChanges = outgoingChangesObs.read(reader); const sessionResource = this.viewModel.activeSessionResourceObs.read(reader); @@ -700,11 +698,7 @@ export class ChangesViewPane extends ViewPane { : { shouldForwardArgs: true }, buttonConfigProvider: (action) => { if (action.id === 'chatEditing.viewChanges' || action.id === 'chatEditing.viewPreviousEdits' || action.id === 'chatEditing.viewAllSessionChanges' || action.id === 'chat.openSessionWorktreeInVSCode') { - const diffStatsLabel = new MarkdownString( - `+${added} -${removed}`, - { supportHtml: true } - ); - return { showIcon: true, showLabel: true, isSecondary: true, customClass: 'working-set-diff-stats', customLabel: diffStatsLabel }; + return { showIcon: true, showLabel: false, isSecondary: true }; } if (action.id === RUN_SESSION_CODE_REVIEW_ACTION_ID) { if (codeReviewLoading) { diff --git a/src/vs/sessions/contrib/changes/browser/media/changesView.css b/src/vs/sessions/contrib/changes/browser/media/changesView.css index 1b187233832..ce34dce6c7c 100644 --- a/src/vs/sessions/contrib/changes/browser/media/changesView.css +++ b/src/vs/sessions/contrib/changes/browser/media/changesView.css @@ -260,21 +260,6 @@ color: var(--vscode-chat-linesRemovedForeground); } -/* Line counts in buttons */ -.changes-view-body .chat-editing-session-actions .monaco-button.working-set-diff-stats { - flex-shrink: 0; - padding-left: 4px; - padding-right: 8px; -} - -.changes-view-body .chat-editing-session-actions .monaco-button .working-set-lines-added { - color: var(--vscode-chat-linesAddedForeground); -} - -.changes-view-body .chat-editing-session-actions .monaco-button .working-set-lines-removed { - color: var(--vscode-chat-linesRemovedForeground); -} - .changes-view-body .chat-editing-session-actions .monaco-button.code-review-comments, .changes-view-body .chat-editing-session-actions .monaco-button.code-review-loading { padding-left: 4px;