Skip to content

Comments

Increase agent session list item height to 48px#295831

Merged
daviddossett merged 1 commit intomainfrom
ddossett/session-list-height
Feb 17, 2026
Merged

Increase agent session list item height to 48px#295831
daviddossett merged 1 commit intomainfrom
ddossett/session-list-height

Conversation

@daviddossett
Copy link
Collaborator

Adds 2px top/bottom padding to agent session list items, increasing the row height from 44px to 48px.

Copilot AI review requested due to automatic review settings February 17, 2026 19:25
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 pull request increases the height of agent session list items from 44px to 48px by adding 2px of vertical padding (top and bottom) to improve the visual spacing and appearance of the list.

Changes:

  • Increased vertical padding in CSS from 4px to 6px (adds 2px top and 2px bottom)
  • Updated the TypeScript ITEM_HEIGHT constant from 44 to 48 to match the new rendered height

Reviewed changes

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

File Description
src/vs/workbench/contrib/chat/browser/agentSessions/media/agentsessionsviewer.css Updated vertical padding from 4px to 6px for agent-session-item
src/vs/workbench/contrib/chat/browser/agentSessions/agentSessionsViewer.ts Updated ITEM_HEIGHT constant from 44 to 48 to reflect the new height

@daviddossett daviddossett marked this pull request as ready for review February 17, 2026 19:29
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 17, 2026
@daviddossett daviddossett enabled auto-merge (squash) February 17, 2026 19:34
@daviddossett daviddossett merged commit b127799 into main Feb 17, 2026
24 checks passed
@daviddossett daviddossett deleted the ddossett/session-list-height branch February 17, 2026 19:55
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.

2 participants