Skip to content

Commit eb7f11a

Browse files
chrisbobbegnprice
authored andcommitted
topic action sheet: Align confirmation dialog text with web app
As suggested by Alya: https://chat.zulip.org/#narrow/stream/48-mobile/topic/confirm.2Fcancel.20modals/near/1401724
1 parent 2021511 commit eb7f11a

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

src/action-sheets/index.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -269,8 +269,11 @@ const deleteTopic = {
269269
action: async ({ streamId, topic, dispatch, _ }) => {
270270
const confirmed = await new Promise((resolve, reject) => {
271271
Alert.alert(
272-
_('Are you sure you want to delete the topic “{topic}”?', { topic }),
273-
_('This will also delete all messages in the topic.'),
272+
_('Delete topic', { topic }),
273+
_(
274+
'Deleting a topic will immediately remove it and its messages for everyone. Other users may find this confusing, especially if they had received an email or push notification related to the deleted messages.\n\nAre you sure you want to permanently delete “{topic}”?',
275+
{ topic },
276+
),
274277
[
275278
{ text: _('Cancel'), onPress: () => resolve(false), style: 'cancel' },
276279
{ text: _('Confirm'), onPress: () => resolve(true), style: 'destructive' },

static/translations/messages_en.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -119,8 +119,7 @@
119119
"Failed to unresolve topic": "Failed to unresolve topic",
120120
"No messages in topic: {streamAndTopic}": "No messages in topic: {streamAndTopic}",
121121
"Delete topic": "Delete topic",
122-
"Are you sure you want to delete the topic “{topic}”?": "Are you sure you want to delete the topic “{topic}”?",
123-
"This will also delete all messages in the topic.": "This will also delete all messages in the topic.",
122+
"Deleting a topic will immediately remove it and its messages for everyone. Other users may find this confusing, especially if they had received an email or push notification related to the deleted messages.\n\nAre you sure you want to permanently delete “{topic}”?": "Deleting a topic will immediately remove it and its messages for everyone. Other users may find this confusing, especially if they had received an email or push notification related to the deleted messages.\n\nAre you sure you want to permanently delete “{topic}”?",
124123
"Unmute topic": "Unmute topic",
125124
"Mute stream": "Mute stream",
126125
"Unmute stream": "Unmute stream",

0 commit comments

Comments
 (0)