Skip to content

Commit f5e8fc0

Browse files
chrisbobbegnprice
authored andcommitted
ui: Remove "?" from confirmation-dialog titles
Discussion: #5439 (comment)
1 parent 8724759 commit f5e8fc0

File tree

5 files changed

+12
-6
lines changed

5 files changed

+12
-6
lines changed

src/account-info/ProfileScreen.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ function LogoutButton(props: {||}) {
9999
onPress={() => {
100100
showConfirmationDialog({
101101
destructive: true,
102-
title: 'Log out?',
102+
title: 'Log out',
103103
message: {
104104
text: 'This will log out {email} on {realmUrl} from the mobile app on this device.',
105105
values: { email: identity.email, realmUrl: identity.realm.toString() },

src/account/AccountPickScreen.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ export default function AccountPickScreen(props: Props): Node {
5555
const { realm, email } = accounts[index];
5656
showConfirmationDialog({
5757
destructive: true,
58-
title: 'Remove account?',
58+
title: 'Remove account',
5959
message: {
6060
text: 'This will make the mobile app on this device forget {email} on {realmUrl}.',
6161
values: { realmUrl: realm.toString(), email },

src/streams/EditStreamCard.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -312,7 +312,7 @@ export default function EditStreamCard(props: Props): Node {
312312

313313
showConfirmationDialog({
314314
destructive: true,
315-
title: 'Discard changes?',
315+
title: 'Discard changes',
316316
message: 'You have unsaved changes. Leave without saving?',
317317
onPressConfirm: () => navigation.dispatch(e.data.action),
318318
_,

src/utils/info.js

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,14 @@ export const showErrorAlert = (
4949

5050
export const showConfirmationDialog = (args: {|
5151
+destructive?: true,
52+
53+
/**
54+
* As in the web app, very brief, sentence case, no question mark
55+
*
56+
* E.g., "Delete topic".
57+
*/
5258
+title: LocalizableText,
59+
5360
+message: LocalizableText,
5461
+learnMoreButton?: LearnMoreButton,
5562
+onPressConfirm: () => void,

static/translations/messages_en.json

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"Confirm": "Confirm",
33
"Configure permissions": "Configure permissions",
44
"You": "You",
5-
"Discard changes?": "Discard changes?",
5+
"Discard changes": "Discard changes",
66
"You have unsaved changes. Leave without saving?": "You have unsaved changes. Leave without saving?",
77
"Who can access the stream?": "Who can access the stream?",
88
"Cannot apply requested settings": "Cannot apply requested settings",
@@ -84,7 +84,6 @@
8484
"Enter": "Enter",
8585
"Switch account": "Switch account",
8686
"Log out": "Log out",
87-
"Log out?": "Log out?",
8887
"This will log out {email} on {realmUrl} from the mobile app on this device.": "This will log out {email} on {realmUrl} from the mobile app on this device.",
8988
"Add new account": "Add new account",
9089
"Search messages": "Search messages",
@@ -318,7 +317,7 @@
318317
"Working remotely": "Working remotely",
319318
"This message was hidden because it is from a user you have muted. Long-press to view.": "This message was hidden because it is from a user you have muted. Long-press to view.",
320319
"Signed out": "Signed out",
321-
"Remove account?": "Remove account?",
320+
"Remove account": "Remove account",
322321
"This will make the mobile app on this device forget {email} on {realmUrl}.": "This will make the mobile app on this device forget {email} on {realmUrl}.",
323322
"Oops! Something went wrong.": "Oops! Something went wrong.",
324323
"Uploading {fileName}...": "Uploading {fileName}...",

0 commit comments

Comments
 (0)