diff --git a/locales/rn/de.json b/locales/rn/de.json index d59235be..0f1d7dec 100644 --- a/locales/rn/de.json +++ b/locales/rn/de.json @@ -868,6 +868,7 @@ "text_signature": "Klartext-Signatur" }, "keywords": { + "delete_confirm_title": "Tag „{label}“ löschen?", "description_mobile": "Farbige Labels zum Organisieren Ihrer E-Mails.", "discover": { "add_all": "Alle hinzufügen", diff --git a/locales/rn/en.json b/locales/rn/en.json index 1297a77e..74f1e4b1 100644 --- a/locales/rn/en.json +++ b/locales/rn/en.json @@ -887,6 +887,7 @@ "text_signature": "Plain-text signature" }, "keywords": { + "delete_confirm_title": "Delete tag \"{label}\"?", "description_mobile": "Colored tags to organize your mail.", "discover": { "add_all": "Add all", diff --git a/src/components/settings/KeywordSettings.tsx b/src/components/settings/KeywordSettings.tsx index e53d7a47..8becb4f5 100644 --- a/src/components/settings/KeywordSettings.tsx +++ b/src/components/settings/KeywordSettings.tsx @@ -45,8 +45,17 @@ export function KeywordSettings() { } }; - const deleteKeyword = (id: string) => { - removeKeyword(id); + // Deleting a tag takes it off every message's tag list on this device; ask + // first, as the reset next to it does. + const deleteKeyword = (id: string, label: string) => { + Alert.alert( + t('settings.keywords.delete_confirm_title', 'Delete tag "{label}"?', { label }), + undefined, + [ + { text: t('common.cancel', 'Cancel'), style: 'cancel' }, + { text: t('settings.keywords.delete', 'Delete tag'), style: 'destructive', onPress: () => removeKeyword(id) }, + ], + ); }; // One stray tap used to wipe a carefully built tag list (webmail removed @@ -119,7 +128,7 @@ export function KeywordSettings() { setEditingId(kw.id)} accessibilityRole="button" accessibilityLabel={t('settings.keywords.edit', "Edit tag")}> - deleteKeyword(kw.id)} accessibilityRole="button" accessibilityLabel={t('settings.keywords.delete', "Delete tag")}> + deleteKeyword(kw.id, kw.label)} accessibilityRole="button" accessibilityLabel={t('settings.keywords.delete', "Delete tag")}>