sessions: changes modal UX polish - remove twistie padding and add header#308338
Merged
sessions: changes modal UX polish - remove twistie padding and add header#308338
Conversation
…ader - Remove left-padding from force-no-twistie in the modal sidebar (scoped to agent-sessions-workbench only) - Add a 'Changes' header with CountBadge to the modal sidebar list Fixes #307705 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
Before
After
|
Contributor
There was a problem hiding this comment.
Pull request overview
Polishes the Changes modal sidebar UX in the Agents (sessions) window by aligning list padding with the main changes list and adding a small header section for clarity.
Changes:
- Override the modal sidebar “force-no-twistie” twistie width to remove the leftover indentation in list mode (scoped to
.agent-sessions-workbench). - Add a “Changes” header row with a
CountBadgeshowing the number of files in the modal sidebar list. - Add CSS styling for the new modal sidebar header.
Show a summary per file
| File | Description |
|---|---|
| src/vs/sessions/browser/media/style.css | Sessions-scoped CSS override to eliminate twistie width/padding in the modal sidebar. |
| src/vs/sessions/contrib/changes/browser/changesView.ts | Adds the modal sidebar header + count badge and adjusts tree layout to account for header height. |
| src/vs/sessions/contrib/changes/browser/media/changesView.css | Styles the new “Changes” header row in the modal sidebar list container. |
Copilot's findings
- Files reviewed: 3/3 changed files
- Comments generated: 1
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
dmitrivMS
approved these changes
Apr 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #307705
Two UX polish fixes for the Changes modal sidebar in the agents window:
1. Remove left-padding from
force-no-twistiein list modeThe twistie element was taking up 10px of width even when hidden, causing misaligned padding compared to the main changes list. Now set to
width: 0— scoped to.agent-sessions-workbenchonly.2. Add a "Changes" header with count badge
Adds a header row above the file list in the modal sidebar, displaying "Changes" with a
CountBadgeshowing the file count. This improves organization and consistency with the right-panel changes list.Files changed
src/vs/sessions/browser/media/style.css— sessions-scoped twistie overridesrc/vs/sessions/contrib/changes/browser/changesView.ts— add header + CountBadge torenderSidebarList()src/vs/sessions/contrib/changes/browser/media/changesView.css— header styling