-
Notifications
You must be signed in to change notification settings - Fork 0
Add refresh icon next to Model label in Chat Settings #176
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
Co-authored-by: tnglemongrass <[email protected]>
| > | ||
| {{ languages.COM_ADD + ' ' + languages.MODEL }} | ||
| </Button> | ||
| <button |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two remarks:
- The ticket says "add refresh icon back to the models list...". Did you check git history, was there such a model refresh button in the past? If yes, how did it look like? Where was it placed?
- Trying out the UI, I think this is not the right place to put that button. It should be placed in the line with the model dropdown. Let's try it next to the "Model" label in the first column.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated in commit b1db539:
-
Git history check: The repository appears to be grafted at commit 9c4b2f5, so I don't have access to earlier history to verify if/how a model refresh button existed previously. The current implementation uses the
i-refreshicon pattern consistent with other refresh buttons in the codebase (e.g., SettingsBasic.vue for preset refresh). -
Positioning updated: Moved the refresh icon next to the "Model" label in the first column of the grid layout as suggested. The icon now appears as:
Model ⟳ [Model Selector ▼]
The refresh icon is wrapped with the label in a flex container within the 120px first column, providing a more compact and intuitive layout that visually associates the refresh action with the model selector.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- good.
- good. Feel free to add that to the PR as well.
Co-authored-by: tnglemongrass <[email protected]>
Description:
Users adding models via the "Add Model" tool had to restart the application to see them in the model selector. Added a refresh icon button next to the "Model" label that calls
models.refreshModels()to scan directories and update the list immediately.Changes Made:
Modified
WebUI/src/components/SettingsChat.vue:useModelsstore and instantiatei-refreshicon (existing SVG asset)models.refreshModels()SettingsBasic.vueTesting Done:
i-refreshicon asset exists and is used elsewhereSettingsBasic.vuemodels.refreshModels()already tested inDownloadDialog.vueScreenshots:
N/A - Adds small icon button next to "Model" label in the settings grid
Checklist:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.