Skip to content

Sessions - remove code that is not needed + increase width for the secondary panel#303586

Merged
lszomoru merged 2 commits intomainfrom
lszomoru/managing-marten
Mar 20, 2026
Merged

Sessions - remove code that is not needed + increase width for the secondary panel#303586
lszomoru merged 2 commits intomainfrom
lszomoru/managing-marten

Conversation

@lszomoru
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings March 20, 2026 18:50
@lszomoru lszomoru enabled auto-merge (squash) March 20, 2026 18:50
@lszomoru lszomoru self-assigned this Mar 20, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.113.0 milestone Mar 20, 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 simplifies the Agent Sessions window by removing a Sessions-specific Git contribution and adjusting the default/minimum sizing of the secondary (auxiliary) panel to be wider.

Changes:

  • Remove src/vs/sessions/contrib/git/browser/git.contribution.ts (Sessions Git sync contribution).
  • Increase the default auxiliary bar size in the Sessions workbench layout from 340 → 380.
  • Increase the auxiliary bar preferred width floor from 340 → 380.

Reviewed changes

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

File Description
src/vs/sessions/contrib/git/browser/git.contribution.ts Removes the Sessions-specific Git sync workbench contribution.
src/vs/sessions/browser/workbench.ts Updates the default grid sizing to allocate more width to the auxiliary bar.
src/vs/sessions/browser/parts/auxiliaryBarPart.ts Updates auxiliary bar preferred width to match the new sizing target.
Comments suppressed due to low confidence (1)

src/vs/sessions/contrib/git/browser/git.contribution.ts:1

  • src/vs/sessions/sessions.desktop.main.ts still imports ./contrib/git/browser/git.contribution.js (around line 212). With this file deleted, the Sessions window build/runtime will hit a module-not-found error. Please remove/update that import (and any remaining references) as part of this deletion.

@lszomoru lszomoru merged commit d8238df into main Mar 20, 2026
19 checks passed
@lszomoru lszomoru deleted the lszomoru/managing-marten branch March 20, 2026 19:31
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