Skip to content

Commit 71a80d7

Browse files
committed
compose: Avoid translating Zulip message destinations
The '#channel > topic' style strings are not supposed to be translated into different languages as they are Zulip's language of expressing the desintation, not something bound to the English language. The string needs to be re-translated in other languages, as the placeholder is now different. See also: zulip#1148 (comment) Signed-off-by: Zixuan James Li <[email protected]>
1 parent 21ee239 commit 71a80d7

12 files changed

+20
-49
lines changed

assets/l10n/app_en.arb

+2-3
Original file line numberDiff line numberDiff line change
@@ -348,12 +348,11 @@
348348
"@composeBoxSelfDmContentHint": {
349349
"description": "Hint text for content input when sending a message to yourself."
350350
},
351-
"composeBoxChannelContentHint": "Message #{channel} > {topic}",
351+
"composeBoxChannelContentHint": "Message {destination}",
352352
"@composeBoxChannelContentHint": {
353353
"description": "Hint text for content input when sending a message to a channel",
354354
"placeholders": {
355-
"channel": {"type": "String", "example": "channel name"},
356-
"topic": {"type": "String", "example": "topic name"}
355+
"destination": {"type": "String", "example": "#channel name > topic name"}
357356
}
358357
},
359358
"composeBoxSendTooltip": "Send",

assets/l10n/app_pl.arb

-14
Original file line numberDiff line numberDiff line change
@@ -263,20 +263,6 @@
263263
"@composeBoxSelfDmContentHint": {
264264
"description": "Hint text for content input when sending a message to yourself."
265265
},
266-
"composeBoxChannelContentHint": "Wiadomość #{channel} > {topic}",
267-
"@composeBoxChannelContentHint": {
268-
"description": "Hint text for content input when sending a message to a channel",
269-
"placeholders": {
270-
"channel": {
271-
"type": "String",
272-
"example": "channel name"
273-
},
274-
"topic": {
275-
"type": "String",
276-
"example": "topic name"
277-
}
278-
}
279-
},
280266
"composeBoxTopicHintText": "Wątek",
281267
"@composeBoxTopicHintText": {
282268
"description": "Hint text for topic input widget in compose box."

assets/l10n/app_ru.arb

-14
Original file line numberDiff line numberDiff line change
@@ -373,20 +373,6 @@
373373
"@composeBoxGenericContentHint": {
374374
"description": "Hint text for content input when sending a message."
375375
},
376-
"composeBoxChannelContentHint": "Сообщение для #{channel} > {topic}",
377-
"@composeBoxChannelContentHint": {
378-
"description": "Hint text for content input when sending a message to a channel",
379-
"placeholders": {
380-
"channel": {
381-
"type": "String",
382-
"example": "channel name"
383-
},
384-
"topic": {
385-
"type": "String",
386-
"example": "topic name"
387-
}
388-
}
389-
},
390376
"composeBoxSendTooltip": "Отправить",
391377
"@composeBoxSendTooltip": {
392378
"description": "Tooltip for send button in compose box."

lib/generated/l10n/zulip_localizations.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -570,8 +570,8 @@ abstract class ZulipLocalizations {
570570
/// Hint text for content input when sending a message to a channel
571571
///
572572
/// In en, this message translates to:
573-
/// **'Message #{channel} > {topic}'**
574-
String composeBoxChannelContentHint(String channel, String topic);
573+
/// **'Message {destination}'**
574+
String composeBoxChannelContentHint(String destination);
575575

576576
/// Tooltip for send button in compose box.
577577
///

lib/generated/l10n/zulip_localizations_ar.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
276276
String get composeBoxSelfDmContentHint => 'Jot down something';
277277

278278
@override
279-
String composeBoxChannelContentHint(String channel, String topic) {
280-
return 'Message #$channel > $topic';
279+
String composeBoxChannelContentHint(String destination) {
280+
return 'Message $destination';
281281
}
282282

283283
@override

lib/generated/l10n/zulip_localizations_en.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
276276
String get composeBoxSelfDmContentHint => 'Jot down something';
277277

278278
@override
279-
String composeBoxChannelContentHint(String channel, String topic) {
280-
return 'Message #$channel > $topic';
279+
String composeBoxChannelContentHint(String destination) {
280+
return 'Message $destination';
281281
}
282282

283283
@override

lib/generated/l10n/zulip_localizations_ja.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
276276
String get composeBoxSelfDmContentHint => 'Jot down something';
277277

278278
@override
279-
String composeBoxChannelContentHint(String channel, String topic) {
280-
return 'Message #$channel > $topic';
279+
String composeBoxChannelContentHint(String destination) {
280+
return 'Message $destination';
281281
}
282282

283283
@override

lib/generated/l10n/zulip_localizations_nb.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
276276
String get composeBoxSelfDmContentHint => 'Jot down something';
277277

278278
@override
279-
String composeBoxChannelContentHint(String channel, String topic) {
280-
return 'Message #$channel > $topic';
279+
String composeBoxChannelContentHint(String destination) {
280+
return 'Message $destination';
281281
}
282282

283283
@override

lib/generated/l10n/zulip_localizations_pl.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
276276
String get composeBoxSelfDmContentHint => 'Zanotuj coś na przyszłość';
277277

278278
@override
279-
String composeBoxChannelContentHint(String channel, String topic) {
280-
return 'Wiadomość #$channel > $topic';
279+
String composeBoxChannelContentHint(String destination) {
280+
return 'Message $destination';
281281
}
282282

283283
@override

lib/generated/l10n/zulip_localizations_ru.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
276276
String get composeBoxSelfDmContentHint => 'Сделать заметку';
277277

278278
@override
279-
String composeBoxChannelContentHint(String channel, String topic) {
280-
return 'Сообщение для #$channel > $topic';
279+
String composeBoxChannelContentHint(String destination) {
280+
return 'Message $destination';
281281
}
282282

283283
@override

lib/generated/l10n/zulip_localizations_sk.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -276,8 +276,8 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
276276
String get composeBoxSelfDmContentHint => 'Jot down something';
277277

278278
@override
279-
String composeBoxChannelContentHint(String channel, String topic) {
280-
return 'Message #$channel > $topic';
279+
String composeBoxChannelContentHint(String destination) {
280+
return 'Message $destination';
281281
}
282282

283283
@override

lib/widgets/compose_box.dart

+2-2
Original file line numberDiff line numberDiff line change
@@ -587,7 +587,7 @@ class _StreamContentInputState extends State<_StreamContentInput> {
587587
destination: TopicNarrow(widget.narrow.streamId, topic),
588588
controller: widget.controller,
589589
hintText: zulipLocalizations.composeBoxChannelContentHint(
590-
streamName, topic.displayName));
590+
'#$streamName > ${topic.displayName}'));
591591
}
592592
}
593593

@@ -646,7 +646,7 @@ class _FixedDestinationContentInput extends StatelessWidget {
646646
final streamName = store.streams[streamId]?.name
647647
?? zulipLocalizations.unknownChannelName;
648648
return zulipLocalizations.composeBoxChannelContentHint(
649-
streamName, topic.displayName);
649+
'#$streamName > ${topic.displayName}');
650650

651651
case DmNarrow(otherRecipientIds: []): // The self-1:1 thread.
652652
return zulipLocalizations.composeBoxSelfDmContentHint;

0 commit comments

Comments
 (0)