From 08af521d1985c6b14520f946e6cdb1ad83a7a709 Mon Sep 17 00:00:00 2001
From: Daniel Michelberger <dml@kq-pwr.de>
Date: Thu, 6 Aug 2026 19:11:38 +0200
Subject: [PATCH] =?UTF-8?q?fix:=20Best=C3=A4tigungsdialog=20und=20Farbvali?=
 =?UTF-8?q?dierung=20bei=20Flag-Verwaltung?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Ersetze native confirm() durch app.confirm.request() mit themingfähigem Dialog
- Erweitere Fehlermeldung beim Löschen mit Hinweis auf Marker-Abhängigkeiten
- Validiere Hex-Farben (#rrggbb) vor dem Speichern
- Konsistenz mit Projektmuster (events, runs, riders, times)
---
 .../dashboard/settings/flags/+page.svelte     | 35 ++++++++++++++-----
 1 file changed, 27 insertions(+), 8 deletions(-)

diff --git a/frontend/src/routes/dashboard/settings/flags/+page.svelte b/frontend/src/routes/dashboard/settings/flags/+page.svelte
index 704283a..5fb1727 100644
--- a/frontend/src/routes/dashboard/settings/flags/+page.svelte
+++ b/frontend/src/routes/dashboard/settings/flags/+page.svelte
@@ -1,5 +1,6 @@
 <script lang="ts">
     import { getTrailFlagContext } from '$lib/stores/trailFlags.svelte'
+    import { app } from '$lib/stores/app.svelte'
     import { Button } from '@/components/ui/button'
     import { Card, CardContent } from '@/components/ui/card'
     import { Badge } from '@/components/ui/badge'
@@ -53,6 +54,11 @@
             return
         }
 
+        if (!/^#[0-9a-fA-F]{6}$/.test(form.color)) {
+            error = 'Bitte eine Farbe im Format #rrggbb angeben.'
+            return
+        }
+
         try {
             if (editing) await flags.edit(editing.id, { ...form } as Parameters<typeof flags.edit>[1])
             else await flags.create({ ...form })
@@ -62,14 +68,27 @@
         }
     }
 
-    async function remove(flag: TrailFlagsResponse) {
-        if (!confirm(`„${flag.label}" wirklich löschen?`)) return
-        try {
-            await flags.remove(flag.id)
-        } catch (e: any) {
-            // Marker verweisen auf den Flag-Typ — PocketBase lehnt das Löschen ab
-            alert(e.message ?? 'Der Flag-Typ konnte nicht gelöscht werden. Vermutlich ist er noch in Verwendung.')
-        }
+    function remove(flag: TrailFlagsResponse) {
+        app.confirm.request({
+            title: 'Flag-Typ löschen?',
+            text: `„${flag.label}" wird entfernt. Marker, die ihn verwenden, verhindern das Löschen.`,
+            yes: async () => {
+                try {
+                    await flags.remove(flag.id)
+                    app.confirm.close()
+                } catch (e: any) {
+                    app.confirm.close()
+                    // Die deleteRule verhindert das Löschen, solange ein Marker
+                    // den Typ verwendet — PocketBase meldet das nur technisch.
+                    alert(
+                        `„${flag.label}" konnte nicht gelöscht werden. ` +
+                        'Vermutlich verwendet ihn noch ein Marker auf einem Trail. ' +
+                        'Entferne zuerst die betroffenen Marker oder markiere sie als erledigt.',
+                    )
+                }
+            },
+            no: () => app.confirm.close(),
+        })
     }
 
     async function seed() {