bug_fix Variantenerzeugung

frontend/src/components/exercises/ExerciseFormPageRoot.jsx

@@ -863,6 +863,33 @@ function ExerciseFormPageRoot() {
     setVariants(rows)
   }, [exerciseId, syncVariantsSavedSnapshot])
 
+  const createVariantFromDraft = useCallback(
+    async ({ showSuccessToast = false } = {}) => {
+      if (!exerciseId) return false
+      if (!variantDraftHasContent(variantDraft)) return true
+      const payload = buildVariantPayloadFromRow(variantDraft)
+      if (payload.variant_name.length < 3) {
+        toast.error('Variantenname mindestens 3 Zeichen')
+        return false
+      }
+      setVariantBusy(true)
+      try {
+        const created = await api.createExerciseVariant(exerciseId, payload)
+        setVariantDraft(emptyVariantDraft())
+        if (created?.id != null) setVariantEditSelection(created.id)
+        await refreshVariants()
+        if (showSuccessToast) toast.success('Variante angelegt.')
+        return true
+      } catch (e) {
+        toast.error(e.message || String(e))
+        return false
+      } finally {
+        setVariantBusy(false)
+      }
+    },
+    [exerciseId, variantDraft, refreshVariants, toast],
+  )
+
   const persistPendingVariantChanges = useCallback(async () => {
     if (!exerciseId) return true
 
@@ -889,28 +916,9 @@ function ExerciseFormPageRoot() {
       }
     }
 
-    if (variantDraftHasContent(variantDraft)) {
+    const draftOk = await createVariantFromDraft()
-      const payload = buildVariantPayloadFromRow(variantDraft)
+    return draftOk
-      if (payload.variant_name.length < 3) {
+  }, [exerciseId, variants, getDirtyVariantRows, refreshVariants, toast, createVariantFromDraft])
-        toast.error('Variantenentwurf: Name mindestens 3 Zeichen, sonst Felder verwerfen oder ausfüllen.')
-        return false
-      }
-      setVariantBusy(true)
-      try {
-        const created = await api.createExerciseVariant(exerciseId, payload)
-        setVariantDraft(emptyVariantDraft())
-        if (created?.id != null) setVariantEditSelection(created.id)
-        await refreshVariants()
-      } catch (e) {
-        toast.error(e.message || String(e))
-        return false
-      } finally {
-        setVariantBusy(false)
-      }
-    }
-
-    return true
-  }, [exerciseId, variantDraft, variants, getDirtyVariantRows, refreshVariants, toast])
 
   const performSaveAttempt = useCallback(
     async ({ fromUnsavedDialog = false, closeAfter = false } = {}) => {
@@ -1227,27 +1235,9 @@ function ExerciseFormPageRoot() {
     }
   }
 
-  const createVariantSubmit = async (e) => {
+  const handleCreateVariantClick = useCallback(async () => {
-    e.preventDefault()
+    await createVariantFromDraft({ showSuccessToast: true })
-    if (!exerciseId) return
+  }, [createVariantFromDraft])
-    const payload = buildVariantPayloadFromRow(variantDraft)
-    if (payload.variant_name.length < 3) {
-      toast.error('Variantenname mindestens 3 Zeichen')
-      return
-    }
-    setVariantBusy(true)
-    try {
-      const created = await api.createExerciseVariant(exerciseId, payload)
-      setVariantDraft(emptyVariantDraft())
-      await refreshVariants()
-      if (created?.id != null) setVariantEditSelection(created.id)
-      else setVariantEditSelection(null)
-    } catch (err) {
-      toast.error(err.message || String(err))
-    } finally {
-      setVariantBusy(false)
-    }
-  }
 
   const selectedVariantForEdit =
     typeof variantEditSelection === 'number' ? variants.find((v) => v.id === variantEditSelection) : null
@@ -2085,9 +2075,8 @@ function ExerciseFormPageRoot() {
             )}
 
             {variantEditSelection === 'new' && (
-              <form
+              <div
                 className="exercise-variant-single-form"
-                onSubmit={createVariantSubmit}
                 style={{ marginTop: '14px', paddingTop: '14px', borderTop: '1px solid var(--border)' }}
               >
                 <h3 style={{ marginTop: 0, fontSize: '1rem' }}>Neue Variante</h3>
@@ -2103,10 +2092,19 @@ function ExerciseFormPageRoot() {
                   linkedExerciseMedia={isEdit ? mediaList : []}
                   onExerciseMediaListChanged={refreshMedia}
                 />
-                <button type="submit" className="btn btn-primary" style={{ marginTop: '10px' }} disabled={variantBusy}>
+                <button
+                  type="button"
+                  className="btn btn-primary"
+                  style={{ marginTop: '10px' }}
+                  disabled={variantBusy}
+                  onClick={handleCreateVariantClick}
+                >
                   {variantBusy ? 'Anlegen…' : 'Variante anlegen'}
                 </button>
-              </form>
+                <p className="exercise-form-panel__hint" style={{ marginTop: '8px', marginBottom: 0 }}>
+                  Alternativ reicht „Speichern“ in der Aktionsleiste — der Entwurf wird dann mitgesichert.
+                </p>
+              </div>
             )}
 
             {selectedVariantForEdit && (