Skip to content
Open
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 16 additions & 7 deletions WebUI/src/components/SettingsChat.vue
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,20 @@
<Label class="whitespace-nowrap">{{ languages.MODEL }}</Label>
<ModelSelector />
</div>
<Button
variant="secondary"
class="self-start w-auto px-3 py-1.5 rounded text-sm"
@click="() => (showModelRequestDialog = true)"
>
{{ languages.COM_ADD + ' ' + languages.MODEL }}
</Button>
<div class="flex items-center gap-2">
<Button
variant="secondary"
class="w-auto px-3 py-1.5 rounded text-sm"
@click="() => (showModelRequestDialog = true)"
>
{{ languages.COM_ADD + ' ' + languages.MODEL }}
</Button>
<button

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two remarks:

  1. 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?
  2. 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.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated in commit b1db539:

  1. 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-refresh icon pattern consistent with other refresh buttons in the codebase (e.g., SettingsBasic.vue for preset refresh).

  2. 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.

@tnglemongrass tnglemongrass Jan 30, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. good.
  2. good. Feel free to add that to the PR as well.

class="svg-icon i-refresh w-5 h-5 text-primary shrink-0"
@click="models.refreshModels"
:title="languages.COM_REFRESH || 'Refresh model list'"
></button>
</div>

<!-- Add Documents button - only shown when RAG is enabled -->
<Button
Expand Down Expand Up @@ -181,8 +188,10 @@ import { usePresets, type ChatPreset } from '@/assets/js/store/presets.ts'
import { usePresetSwitching } from '@/assets/js/store/presetSwitching.ts'
import PresetSelector from '@/components/PresetSelector.vue'
import * as toast from '@/assets/js/toast'
import { useModels } from '@/assets/js/store/models.ts'

const showModelRequestDialog = ref(false)
const models = useModels()
const showUploader = ref(false)
const processing = ref(false)
const i18nState = useI18N().state
Expand Down