Skip to content

Use settings icon instead of tools icon in chat input#299068

Merged
daviddossett merged 1 commit intomainfrom
daviddossett/settings-icon-chat-input
Mar 3, 2026
Merged

Use settings icon instead of tools icon in chat input#299068
daviddossett merged 1 commit intomainfrom
daviddossett/settings-icon-chat-input

Conversation

@daviddossett
Copy link
Collaborator

Use the settings codicon instead of tools for the Configure Tools button in the chat input toolbar.

Copilot AI review requested due to automatic review settings March 3, 2026 21:48
@daviddossett daviddossett marked this pull request as ready for review March 3, 2026 21:49
@vs-code-engineering vs-code-engineering bot added this to the 1.111.0 milestone Mar 3, 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

Updates the chat input “Configure Tools” toolbar button to use the settings codicon instead of tools, aligning both the action icon and the CSS that targets it in the collapsed toolbar state.

Changes:

  • Switched ConfigureToolsAction to use Codicon.settings.
  • Updated chat input toolbar CSS to hide the button based on .codicon-settings when chevrons are hidden.

Reviewed changes

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

File Description
src/vs/workbench/contrib/chat/browser/widget/media/chat.css Updates the collapsed-toolbar selector to target .codicon-settings instead of .codicon-tools.
src/vs/workbench/contrib/chat/browser/actions/chatToolActions.ts Changes the Configure Tools... action icon from Codicon.tools to Codicon.settings.

@daviddossett daviddossett merged commit 2910902 into main Mar 3, 2026
24 checks passed
@daviddossett daviddossett deleted the daviddossett/settings-icon-chat-input branch March 3, 2026 22:50
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