Skip to content

sessions - fix title bar issue on macOS#303224

Merged
bpasero merged 3 commits intomainfrom
ben/anonymous-mule
Mar 19, 2026
Merged

sessions - fix title bar issue on macOS#303224
bpasero merged 3 commits intomainfrom
ben/anonymous-mule

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Mar 19, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 19, 2026 15:24
@bpasero bpasero enabled auto-merge (squash) March 19, 2026 15:24
@bpasero bpasero self-assigned this Mar 19, 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 updates the Agent Sessions titlebar implementation to address a macOS-specific Electron issue where the Sessions window may not appear in the system “Windows” menu due to the initial document.title value.

Changes:

  • Inject IProductService into Sessions titlebar parts so the product’s native window title can be applied consistently.
  • Add a macOS title “toggle” workaround in the Sessions browser titlebar to force macOS/Electron to recognize the window in the “Windows” menu.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/vs/sessions/electron-browser/parts/titlebarPart.ts Threads IProductService through native titlebar part constructors and super calls.
src/vs/sessions/browser/parts/titlebarPart.ts Injects IProductService and applies a macOS document.title workaround during titlebar creation.

@bpasero bpasero added this to the 1.113.0 milestone Mar 19, 2026
@bpasero bpasero merged commit 08316e4 into main Mar 19, 2026
19 checks passed
@bpasero bpasero deleted the ben/anonymous-mule branch March 19, 2026 17:29
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