Skip to content

Conversation

@kearns-cu
Copy link
Contributor

@kearns-cu kearns-cu commented Jan 20, 2026

Summary

  • add a manage models icon next to the model selector search
  • close the popover and open the manage models dialog on click
  • expose a list search action slot for inline controls

Testing

  • Tested on windows

Screenshots

Now

Cap 2026-01-20 at 18 43 21

Screenshot 2026-01-20 183656

Before

image

Fixes #9720
Requested by Adam (Figma: https://www.figma.com/design/nm7fT1he2hcniHIipY1iEk/opencode-open-source?node-id=62-15981&t=FSAXJprvW4rRgLZU-0)

@github-actions
Copy link
Contributor

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@github-actions
Copy link
Contributor

The following comment was made by an LLM, it may be inaccurate:

No duplicate PRs found

@kearns-cu kearns-cu changed the title feat(app): add manage models icon to selector feat(desktop): add manage models icon to selector Jan 20, 2026
@kearns-cu kearns-cu changed the title feat(desktop): add manage models icon to selector feat(app): add manage models icon to selector Jan 20, 2026
@kearns-cu kearns-cu changed the title feat(app): add manage models icon to selector feat(app): add manage models icon to selector (per Figma request) Jan 20, 2026
@kearns-cu kearns-cu marked this pull request as ready for review January 20, 2026 23:52
@kearns-cu
Copy link
Contributor Author

The nix desktop CI failure on macos-15-intel is unrelated to my changes. The error is from upstream nixpkgs package models-dev-0-unstable-2026-01-14 not supporting x86_64-darwin. This PR only touches UI code.

@adamdotdevin
Copy link
Contributor

I think he had the button outside of the search input, right? @kearns-cu

@kearns-cu kearns-cu marked this pull request as draft January 21, 2026 04:51
@kearns-cu
Copy link
Contributor Author

Right! Will adjust that @adamdotdevin

@kearns-cu
Copy link
Contributor Author

Sorted @adamdotdevin
Cap 2026-01-21 at 00 18 03

@kearns-cu kearns-cu marked this pull request as ready for review January 21, 2026 05:25
@mohamedbouddi7777-dev
Copy link

mohamedbouddi7777-dev commented Jan 21, 2026 via email

@kearns-cu kearns-cu force-pushed the feature/manage-models-icon branch 2 times, most recently from ea1cfb3 to 241b856 Compare January 21, 2026 06:40
@kearns-cu
Copy link
Contributor Author

Resolved merge conflicts on github. Initially did it locally so ignore my undo force push lol. Nice to see your language support code come in! 谢谢

@adamdotdevin adamdotdevin merged commit 996eeb1 into anomalyco:dev Jan 21, 2026
7 of 8 checks passed
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.

Desktop: add manage models shortcut to model selector

3 participants