Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Mar 19, 2025

Context

I found a few themes where we need vscode-list-activeSelectionForeground.

Implementation

Screenshots

before after

How to Test

Get in Touch


Important

Updated VSCode theme for select, dropdown, and command components, including CSS and component refactoring for consistent styling.

  • Theme Updates:
    • Updated ModelPicker.tsx, select-dropdown.tsx, and dropdown-menu.tsx to use vscode-list-activeSelectionForeground for active selections.
    • Adjusted CSS classes in autosize-textarea.tsx, button.tsx, and command.tsx for consistent styling.
    • Modified index.css to include --color-vscode-list-activeSelectionForeground.
  • Component Changes:
    • Changed Button variant to combobox in ModelPicker.tsx.
    • Removed DropdownMenuSub and related components from dropdown-menu.tsx.
  • Testing:
    • Updated select-dropdown.test.tsx to reflect changes in dropdown item rendering and selection logic.

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

@changeset-bot
Copy link

changeset-bot bot commented Mar 19, 2025

🦋 Changeset detected

Latest commit: 84dd8a5

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 19, 2025
@dosubot dosubot bot added the Enhancement New feature or request label Mar 19, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 19, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Mar 19, 2025
@cte cte merged commit fbb5cfc into main Mar 19, 2025
9 checks passed
@cte cte deleted the cte/more-theme-fixes branch March 19, 2025 05:07
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Mar 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants