Skip to content

Agents - move section chevron to the left and reveal it on hover - #331545

Merged
Ladislau Szomoru (lszomoru) merged 1 commit into
mainfrom
lszomoru/constitutional-egret
Aug 19, 2026
Merged

Ladislau Szomoru (lszomoru) merged 1 commit into
mainfrom
lszomoru/constitutional-egret

Conversation

@lszomoru

Copy link
Copy Markdown
Member

No description provided.

Copilot AI balanced review requested due to automatic review settings August 18, 2026 19:37

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

Moves session-section chevrons left and reveals them on hover or keyboard focus.

Changes:

  • Adds icons for section and group headers.
  • Swaps icons for chevrons during interaction.
  • Updates toolbar spacing and documentation.
Show a summary per file
File Description
src/vs/sessions/SESSIONS_LIST.md Clarifies chevron focus behavior.
src/vs/sessions/contrib/sessions/browser/views/sessionsList.ts Reorders chevrons and adds header icons.
src/vs/sessions/contrib/sessions/browser/media/sessionsList.css Styles reveal behavior and toolbar spacing.

Review details

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 3/3 changed files
  • Comments generated: 2
  • Review effort level: Balanced

Comment thread src/vs/sessions/contrib/sessions/browser/media/sessionsList.css
Comment thread src/vs/sessions/contrib/sessions/browser/media/sessionsList.css
@github-actions

Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: d9926423 Current: bff8ddb6

Changed (10)

sessions/sessionsList/SessionsList_CustomGroup/Dark
Before After
before after
sessions/sessionsList/SessionsList_CustomGroup/Light
Before After
before after
sessions/sessionsList/SessionsList_CustomGroup_LongWorkspaceNarrow/Dark
Before After
before after
sessions/sessionsList/SessionsList_CustomGroup_LongWorkspaceNarrow/Light
Before After
before after
sessions/sessionsList/SessionsList_CustomGroup_InProgress/Dark
Before After
before after
sessions/sessionsList/SessionsList_CustomGroup_InProgress/Light
Before After
before after
sessions/sessionsList/SessionsList_WorkspaceSection/Dark
Before After
before after
sessions/sessionsList/SessionsList_WorkspaceSection/Light
Before After
before after
sessions/sessionsList/SessionsList_CustomGroup_Phone/Dark
Before After
before after
sessions/sessionsList/SessionsList_CustomGroup_Phone/Light
Before After
before after

@lszomoru
Ladislau Szomoru (lszomoru) merged commit 474a349 into main Aug 19, 2026
62 of 64 checks passed
@lszomoru
Ladislau Szomoru (lszomoru) deleted the lszomoru/constitutional-egret branch August 19, 2026 08:33
@vs-code-engineering vs-code-engineering Bot added this to the 1.135.0 milestone Aug 19, 2026
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.

4 participants

Sponsor
SponsoredKunjungi sekarang
Promo