Add right click visibility toggle for sign in button#315022
Merged
Merged
Conversation
Co-authored-by: Copilot <copilot@github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds a configurable right-click toggle for the Copilot “Sign In” title bar button, aligning it with other title bar context menu visibility controls and ensuring the status bar presentation stays consistent when the title bar affordance is hidden.
Changes:
- Introduces a new setting,
chat.titleBar.signIn.enabled, to control whether the signed-out “Sign In” affordance appears in the title bar or falls back to the status bar. - Adds a title bar context menu toggle action (
ToggleSignInTitleBarAction) for that setting, and updates the title bar “Sign In” action’s visibility condition to respect it. - Updates the Copilot status bar entry to hide the “Sign In” label when the title bar affordance is visible and to react to relevant context/config changes.
Show a summary per file
| File | Description |
|---|---|
| src/vs/workbench/contrib/chat/common/constants.ts | Adds a new ChatConfiguration enum entry for the title bar sign-in visibility setting. |
| src/vs/workbench/contrib/chat/browser/chatStatus/chatStatusEntry.ts | Updates status bar “Sign In” label logic and adds centralized visibility computation based on context/config. |
| src/vs/workbench/contrib/chat/browser/chatSetup/chatSetupContributions.ts | Adds a title bar context menu toggle action and gates title bar “Sign In” button visibility by the new setting. |
| src/vs/workbench/contrib/chat/browser/chat.contribution.ts | Registers the new configuration setting and its description/default. |
Copilot's findings
- Files reviewed: 4/4 changed files
- Comments generated: 2
| if (isWeb) { | ||
| return false; | ||
| } | ||
|
|
Comment on lines
+109
to
+113
| this._register(this.contextKeyService.onDidChangeContext(e => { | ||
| if (e.affectsSome(new Set(['updateTitleBar', InEditorZenModeContext.key]))) { | ||
| this.update(); | ||
| } | ||
| })); |
lramos15
approved these changes
May 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds right click toggle for Sign in button to align with other elements in the title bar. Currently this can be hidden by signing in or disabling AI features so we want to support right click hide / show as well
Configuration and UI improvements:
chat.titleBar.signIn.enabledto control whether the Copilot "Sign In" button appears in the title bar or falls back to the status bar when disabled. [1] [2]ToggleSignInTitleBarAction, which allows users to toggle the visibility of the title bar sign-in button via the command palette. [1] [2] [3]isSignInTitleBarAffordanceVisiblemethod to centralize the logic for determining when the title bar sign-in button should be shown, considering web context, workspace settings, Zen mode, and the new configuration.