Skip to content

update model picker icons - #327056

Merged
justschen merged 2 commits into
mainfrom
justin/barraskewda
Jul 23, 2026
Merged

update model picker icons#327056
justschen merged 2 commits into
mainfrom
justin/barraskewda

Conversation

@justschen

Copy link
Copy Markdown
Collaborator

updates generic icon with new icons (or no icons)

Copilot AI review requested due to automatic review settings July 22, 2026 20:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates model picker branding with provider-specific icons and hides redundant icons in harness-backed sessions.

Changes:

  • Adds Gemini, Kimi, and Microsoft icon selection.
  • Uses generic icons for BYOK models.
  • Replaces generic-icon configuration with icon visibility controls.
Show a summary per file
File Description
modelProviderIcons.test.ts Tests provider and BYOK icon selection.
modelProviderIcons.ts Registers and resolves provider icons.
modelPickerWidget.ts Controls selected-model icon visibility.
modelPickerActionItem.ts Renames the presentation option.
chatPhoneInputPresenter.ts Applies icon visibility on mobile.
chatInputPart.ts Hides redundant harness model icons.
modelPicker.ts Enables icons in the Agents window.

Review details

  • Files reviewed: 7/7 changed files
  • Comments generated: 1
  • Review effort level: Medium

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@justschen
justschen marked this pull request as ready for review July 22, 2026 21:59
@justschen
justschen enabled auto-merge (squash) July 22, 2026 22:19
@justschen
justschen merged commit d252c9e into main Jul 23, 2026
29 checks passed
@justschen
justschen deleted the justin/barraskewda branch July 23, 2026 00:29
@vs-code-engineering vs-code-engineering Bot added this to the 1.131.0 milestone Jul 23, 2026
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