feat(desktop): enhanced model menu with search, grouping by provider, and adaptive width#4139
feat(desktop): enhanced model menu with search, grouping by provider, and adaptive width#4139ttmouse wants to merge 2 commits into
Conversation
ttmouse
commented
Jun 12, 2026
ttmouse
commented
- Simplified ModelSwitcher to match upstream (removed closing animation)
- Added search input with auto-focus for filtering models by name/provider
- Grouped models by provider with accent-colored group labels and Brain icon
- Adjusted width to adapt to content (min triggerWidth, max min(90vw, 480px))
- Added max-height: 50vh with scroll for large model lists
- Fixed empty-state duplication, i18n placeholder, and stale group keys
… and adaptive width - Simplified ModelSwitcher to match upstream (removed closing animation) - Added search input with auto-focus for filtering models by name/provider - Grouped models by provider with accent-colored group labels and Brain icon - Adjusted width to adapt to content (min triggerWidth, max min(90vw, 480px)) - Added max-height: 50vh with scroll for large model lists
|
github-actions
Bot
added
v2
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: edc45dee38
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| * -------------------------------------------------------------------------- */ | ||
|
|
||
| opacity: 1; | ||
| } |
There was a problem hiding this comment.
This unmatched } makes the frontend CSS syntax check fail (node scripts/check-css-syntax.mjs src/styles.css reports Found a closing brace without a matching opening brace), and desktop/frontend/package.json runs that check at the start of npm run build, so any desktop frontend build with this commit will stop before typechecking or bundling.
Useful? React with 👍 / 👎.
1 participant
