Refine AI button labeling logic: only use 'Translate' for same-field cross-language content

This commit is contained in:
Christian Vidal Wolf
2026-04-20 16:53:21 +02:00
parent 4c39c9ec53
commit 67a0466627
+8 -3
View File
@@ -428,7 +428,7 @@ export function EditPanel({ row, rowIndex, onSave, onClose, onCaptureState }: Ed
field="shortDe" field="shortDe"
value={formData.shortDe} value={formData.shortDe}
isModified={isModified('shortDe')} isModified={isModified('shortDe')}
canTranslate={!!formData.shortEn || !!formData.longDe} canTranslate={!!formData.shortEn}
isGenerated={generatedFields.has('shortDe')} isGenerated={generatedFields.has('shortDe')}
isLoading={loadingField === 'shortDe'} isLoading={loadingField === 'shortDe'}
onGenerate={() => handleGenerate('shortDe')} onGenerate={() => handleGenerate('shortDe')}
@@ -440,7 +440,7 @@ export function EditPanel({ row, rowIndex, onSave, onClose, onCaptureState }: Ed
field="shortEn" field="shortEn"
value={formData.shortEn} value={formData.shortEn}
isModified={isModified('shortEn')} isModified={isModified('shortEn')}
canTranslate={!!formData.shortDe || !!formData.longEn} canTranslate={!!formData.shortDe}
isGenerated={generatedFields.has('shortEn')} isGenerated={generatedFields.has('shortEn')}
isLoading={loadingField === 'shortEn'} isLoading={loadingField === 'shortEn'}
onGenerate={() => handleGenerate('shortEn')} onGenerate={() => handleGenerate('shortEn')}
@@ -479,7 +479,12 @@ export function EditPanel({ row, rowIndex, onSave, onClose, onCaptureState }: Ed
onConfirm={executeGenerate} onConfirm={executeGenerate}
onCancel={() => setPendingGeminiField(null)} onCancel={() => setPendingGeminiField(null)}
title="Confirm AI Action" title="Confirm AI Action"
message={`Are you sure you want to use Gemini to ${((pendingGeminiField === 'longEn' && formData.longDe) || (pendingGeminiField === 'shortEn' && formData.shortDe)) ? 'translate' : 'generate'} the ${pendingGeminiField} field?`} message={`Are you sure you want to use Gemini to ${
((pendingGeminiField === 'shortEn' && formData.shortDe) ||
(pendingGeminiField === 'shortDe' && formData.shortEn) ||
(pendingGeminiField === 'longEn' && formData.longDe) ||
(pendingGeminiField === 'longDe' && formData.longEn))
? 'translate' : 'generate'} the ${pendingGeminiField} field?`}
type="info" type="info"
confirmText="Start Generation" confirmText="Start Generation"
/> />