Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Mar 9, 2025

Context

#1497

Implementation

Same approach as #1484

Screenshots

before after
Screenshot 2025-03-09 at 9 17 55 AM Screenshot 2025-03-09 at 9 16 48 AM

How to Test

Open the VS Code LM API model picker in VS Code and see if it goes over or under the header.


Important

Replaces Dropdown with Select for VS Code LM model picker in ApiOptions.tsx, improving model selection handling.

  • Behavior:
    • Replaces Dropdown with Select for VS Code LM model picker in ApiOptions.tsx.
    • Updates onChange to onValueChange for handling model selection.
  • Misc:
    • Adds changeset eleven-birds-doubt.md for patch update.

This description was created by Ellipsis for 68cfdc9. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Mar 9, 2025

🦋 Changeset detected

Latest commit: 68cfdc9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
roo-cline Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 9, 2025
@mrubens mrubens force-pushed the fix_vscode_lm_dropdown branch 2 times, most recently from c1304f6 to e5b256d Compare March 9, 2025 13:35
@mrubens mrubens force-pushed the fix_vscode_lm_dropdown branch from e5b256d to 68cfdc9 Compare March 9, 2025 13:39
@mrubens mrubens merged commit a1c6387 into main Mar 9, 2025
9 checks passed
@mrubens mrubens deleted the fix_vscode_lm_dropdown branch March 9, 2025 13:45
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Mar 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants