Skip to content

Close welcome view when clicking on View all sessions#290069

Merged
osortega merged 1 commit intomainfrom
osortega/substantial-skunk
Jan 24, 2026
Merged

Close welcome view when clicking on View all sessions#290069
osortega merged 1 commit intomainfrom
osortega/substantial-skunk

Conversation

@osortega
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 24, 2026 01:02
@osortega osortega self-assigned this Jan 24, 2026
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 24, 2026
}

private revealMaximizedChat(): void {
this.commandService.executeCommand('workbench.action.closeActiveEditor');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't we be awaiting these to ensure they don't step on each others' feet? Also try to use proper services rather than executeCommand, since it's untyped, relies on a magic string, no "find all references" etc.

Copy link
Copy Markdown
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

Updates the agent sessions welcome experience so the welcome editor closes when navigating to the full chat/sessions experience.

Changes:

  • Close the active editor before opening the Chat view from the welcome page.
  • Keep existing behavior that opens Chat and maximizes the auxiliary bar when needed.

@osortega osortega merged commit c4eecb7 into main Jan 24, 2026
27 of 28 checks passed
@osortega osortega deleted the osortega/substantial-skunk branch January 24, 2026 02:37
osortega added a commit that referenced this pull request Jan 25, 2026
Close welcome view when clicking on View all sessions
@vs-code-engineering vs-code-engineering bot locked and limited conversation to collaborators Mar 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants