From 1107ce1af16b046212452a28f4d9a6d7aea3b1c6 Mon Sep 17 00:00:00 2001
From: salvacybersec <salva@opsecti.local>
Date: Mon, 10 Nov 2025 21:21:29 +0300
Subject: [PATCH] feat: Improve Ollama settings UX

- Auto-save settings before testing connection
- Make model list clickable for easy selection
- Models auto-populate when clicked
- Better user experience for Ollama configuration

Now users can click on a model from the list to select it instantly.
---
 frontend/src/pages/Settings.jsx | 22 ++++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/frontend/src/pages/Settings.jsx b/frontend/src/pages/Settings.jsx
index 30ed8ed..2b1e5a6 100644
--- a/frontend/src/pages/Settings.jsx
+++ b/frontend/src/pages/Settings.jsx
@@ -148,6 +148,14 @@ function Settings() {
     setAlerts({ ...alerts, ollama: null });
 
     try {
+      // First save the settings if they are filled
+      if (settings.ollama_server_url && settings.ollama_model) {
+        await axios.put(`${API_URL}/api/ollama/settings`, {
+          ollama_server_url: settings.ollama_server_url,
+          ollama_model: settings.ollama_model,
+        }, { withCredentials: true });
+      }
+
       const response = await axios.get(
         `${API_URL}/api/ollama/test`,
         { withCredentials: true }
@@ -425,10 +433,20 @@ function Settings() {
             {ollamaModels.length > 0 && (
               <Alert severity="info" sx={{ mt: 2 }}>
                 <Typography variant="body2" fontWeight="bold">
-                  Mevcut Modeller:
+                  Mevcut Modeller (tıklayarak seçin):
                 </Typography>
                 {ollamaModels.map((model, idx) => (
-                  <Typography key={idx} variant="body2">
+                  <Typography 
+                    key={idx} 
+                    variant="body2"
+                    sx={{ 
+                      cursor: 'pointer', 
+                      '&:hover': { backgroundColor: 'action.hover' },
+                      p: 0.5,
+                      borderRadius: 1,
+                    }}
+                    onClick={() => setSettings({ ...settings, ollama_model: model.name })}
+                  >
                     • {model.name} ({(model.size / 1024 / 1024 / 1024).toFixed(1)} GB)
                   </Typography>
                 ))}