action_sheet: Add "Copy link to channel" button

sm-sayedi · sm-sayedi · commit 373cfb7b1451 · 2025-07-29T20:42:23.000+04:30
Fixes: zulip#1227
diff --git a/assets/l10n/app_en.arb b/assets/l10n/app_en.arb
@@ -100,6 +100,10 @@
   "@actionSheetOptionMarkChannelAsRead": {
     "description": "Label for marking a channel as read."
   },
+  "actionSheetOptionCopyChannelLink": "Copy link to channel",
+  "@actionSheetOptionCopyChannelLink": {
+    "description": "Label for copy channel link button on action sheet."
+  },
   "actionSheetOptionListOfTopics": "List of topics",
   "@actionSheetOptionListOfTopics": {
     "description": "Label for navigating to a channel's topic-list page."
@@ -357,6 +361,10 @@
   "@successMessageLinkCopied": {
     "description": "Message when link of a message was copied to the user's system clipboard."
   },
+  "successChannelLinkCopied": "Channel link copied",
+  "@successChannelLinkCopied": {
+    "description": "Message when link of a channel was copied to the user's system clipboard."
+  },
   "errorBannerDeactivatedDmLabel": "You cannot send messages to deactivated users.",
   "@errorBannerDeactivatedDmLabel": {
     "description": "Label text for error banner when sending a message to one or multiple deactivated users."
diff --git a/lib/generated/l10n/zulip_localizations.dart b/lib/generated/l10n/zulip_localizations.dart
@@ -281,6 +281,12 @@ abstract class ZulipLocalizations {
   /// **'Mark channel as read'**
   String get actionSheetOptionMarkChannelAsRead;
 
+  /// Label for copy channel link button on action sheet.
+  ///
+  /// In en, this message translates to:
+  /// **'Copy link to channel'**
+  String get actionSheetOptionCopyChannelLink;
+
   /// Label for navigating to a channel's topic-list page.
   ///
   /// In en, this message translates to:
@@ -619,6 +625,12 @@ abstract class ZulipLocalizations {
   /// **'Message link copied'**
   String get successMessageLinkCopied;
 
+  /// Message when link of a channel was copied to the user's system clipboard.
+  ///
+  /// In en, this message translates to:
+  /// **'Channel link copied'**
+  String get successChannelLinkCopied;
+
   /// Label text for error banner when sending a message to one or multiple deactivated users.
   ///
   /// In en, this message translates to:
diff --git a/lib/generated/l10n/zulip_localizations_ar.dart b/lib/generated/l10n/zulip_localizations_ar.dart
@@ -90,6 +90,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Mark channel as read';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'List of topics';
 
@@ -300,6 +303,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Message link copied';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'You cannot send messages to deactivated users.';
diff --git a/lib/generated/l10n/zulip_localizations_de.dart b/lib/generated/l10n/zulip_localizations_de.dart
@@ -92,6 +92,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {
   String get actionSheetOptionMarkChannelAsRead =>
       'Kanal als gelesen markieren';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'Themenliste';
 
@@ -313,6 +316,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Nachrichtenlink kopiert';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'Du kannst keine Nachrichten an deaktivierte Nutzer:innen senden.';
diff --git a/lib/generated/l10n/zulip_localizations_en.dart b/lib/generated/l10n/zulip_localizations_en.dart
@@ -90,6 +90,9 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Mark channel as read';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'List of topics';
 
@@ -300,6 +303,9 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Message link copied';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'You cannot send messages to deactivated users.';
diff --git a/lib/generated/l10n/zulip_localizations_fr.dart b/lib/generated/l10n/zulip_localizations_fr.dart
@@ -90,6 +90,9 @@ class ZulipLocalizationsFr extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Mark channel as read';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'List of topics';
 
@@ -300,6 +303,9 @@ class ZulipLocalizationsFr extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Message link copied';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'You cannot send messages to deactivated users.';
diff --git a/lib/generated/l10n/zulip_localizations_it.dart b/lib/generated/l10n/zulip_localizations_it.dart
@@ -91,6 +91,9 @@ class ZulipLocalizationsIt extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Segna il canale come letto';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'Elenco degli argomenti';
 
@@ -310,6 +313,9 @@ class ZulipLocalizationsIt extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Collegamento messaggio copiato';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'Non è possibile inviare messaggi agli utenti disattivati.';
diff --git a/lib/generated/l10n/zulip_localizations_ja.dart b/lib/generated/l10n/zulip_localizations_ja.dart
@@ -89,6 +89,9 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'チャンネルを既読にする';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'トピック一覧';
 
@@ -297,6 +300,9 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Message link copied';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'You cannot send messages to deactivated users.';
diff --git a/lib/generated/l10n/zulip_localizations_nb.dart b/lib/generated/l10n/zulip_localizations_nb.dart
@@ -90,6 +90,9 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Mark channel as read';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'List of topics';
 
@@ -300,6 +303,9 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Message link copied';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'You cannot send messages to deactivated users.';
diff --git a/lib/generated/l10n/zulip_localizations_pl.dart b/lib/generated/l10n/zulip_localizations_pl.dart
@@ -92,6 +92,9 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
   String get actionSheetOptionMarkChannelAsRead =>
       'Oznacz kanał jako przeczytany';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'Lista wątków';
 
@@ -308,6 +311,9 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Skopiowano odnośnik wiadomości';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'Nie można wysyłać wiadomości do dezaktywowanych użytkowników.';
diff --git a/lib/generated/l10n/zulip_localizations_ru.dart b/lib/generated/l10n/zulip_localizations_ru.dart
@@ -92,6 +92,9 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
   String get actionSheetOptionMarkChannelAsRead =>
       'Отметить канал как прочитанный';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'Список тем';
 
@@ -309,6 +312,9 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Ссылка на сообщение скопирована';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'Нельзя отправить сообщение отключенным пользователям.';
diff --git a/lib/generated/l10n/zulip_localizations_sk.dart b/lib/generated/l10n/zulip_localizations_sk.dart
@@ -90,6 +90,9 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Mark channel as read';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'List of topics';
 
@@ -300,6 +303,9 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Message link copied';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'You cannot send messages to deactivated users.';
diff --git a/lib/generated/l10n/zulip_localizations_sl.dart b/lib/generated/l10n/zulip_localizations_sl.dart
@@ -90,6 +90,9 @@ class ZulipLocalizationsSl extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Označi kanal kot prebran';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'Seznam tem';
 
@@ -320,6 +323,9 @@ class ZulipLocalizationsSl extends ZulipLocalizations {
   String get successMessageLinkCopied =>
       'Povezava do sporočila je bila kopirana';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'Deaktiviranim uporabnikom ne morete pošiljati sporočil.';
diff --git a/lib/generated/l10n/zulip_localizations_uk.dart b/lib/generated/l10n/zulip_localizations_uk.dart
@@ -93,6 +93,9 @@ class ZulipLocalizationsUk extends ZulipLocalizations {
   String get actionSheetOptionMarkChannelAsRead =>
       'Позначити канал як прочитаний';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'Список тем';
 
@@ -311,6 +314,9 @@ class ZulipLocalizationsUk extends ZulipLocalizations {
   String get successMessageLinkCopied =>
       'Посилання на повідомлення скопійовано';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'Ви не можете надсилати повідомлення деактивованим користувачам.';
diff --git a/lib/generated/l10n/zulip_localizations_zh.dart b/lib/generated/l10n/zulip_localizations_zh.dart
@@ -90,6 +90,9 @@ class ZulipLocalizationsZh extends ZulipLocalizations {
   @override
   String get actionSheetOptionMarkChannelAsRead => 'Mark channel as read';
 
+  @override
+  String get actionSheetOptionCopyChannelLink => 'Copy link to channel';
+
   @override
   String get actionSheetOptionListOfTopics => 'List of topics';
 
@@ -300,6 +303,9 @@ class ZulipLocalizationsZh extends ZulipLocalizations {
   @override
   String get successMessageLinkCopied => 'Message link copied';
 
+  @override
+  String get successChannelLinkCopied => 'Channel link copied';
+
   @override
   String get errorBannerDeactivatedDmLabel =>
       'You cannot send messages to deactivated users.';
diff --git a/lib/widgets/action_sheet.dart b/lib/widgets/action_sheet.dart
@@ -205,6 +205,9 @@ void showChannelActionSheet(BuildContext context, {
       MarkChannelAsReadButton(pageContext: pageContext, channelId: channelId));
   }
 
+  optionButtons.add(
+    CopyChannelLinkButton(channelId: channelId, pageContext: pageContext));
+
   _showActionSheet(pageContext, optionButtons: optionButtons);
 }
 
@@ -256,6 +259,34 @@ class MarkChannelAsReadButton extends ActionSheetMenuItemButton {
   }
 }
 
+class CopyChannelLinkButton extends ActionSheetMenuItemButton {
+  const CopyChannelLinkButton({
+    super.key,
+    required this.channelId,
+    required super.pageContext,
+  });
+
+  final int channelId;
+
+  @override
+  IconData get icon => ZulipIcons.link;
+
+  @override
+  String label(ZulipLocalizations zulipLocalizations) {
+    return zulipLocalizations.actionSheetOptionCopyChannelLink;
+  }
+
+  @override
+  void onPressed() async {
+    final localizations = ZulipLocalizations.of(pageContext);
+    final store = PerAccountStoreWidget.of(pageContext);
+
+    PlatformActions.copyWithPopup(context: pageContext,
+      successContent: Text(localizations.successChannelLinkCopied),
+      data: ClipboardData(text: narrowLink(store, ChannelNarrow(channelId)).toString()));
+  }
+}
+
 /// Show a sheet of actions you can take on a topic.
 ///
 /// Needs a [PageRoot] ancestor.
diff --git a/test/widgets/action_sheet_test.dart b/test/widgets/action_sheet_test.dart
@@ -243,6 +243,7 @@ void main() {
         check(actionSheetFinder).findsOne();
         checkButton('List of topics');
         checkButton('Mark channel as read');
+        checkButton('Copy link to channel');
       }
 
       testWidgets('show from inbox', (tester) async {
@@ -342,6 +343,32 @@ void main() {
           expectedTitle: "Mark as read failed");
       });
     });
+
+    group('CopyChannelLinkButton', () {
+      setUp(() async {
+        TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger.setMockMethodCallHandler(
+          SystemChannels.platform,
+          MockClipboard().handleMethodCall,
+        );
+      });
+
+      Future<void> tapCopyChannelLinkButton(WidgetTester tester) async {
+        await tester.ensureVisible(find.byIcon(ZulipIcons.link, skipOffstage: false));
+        await tester.tap(find.byIcon(ZulipIcons.link));
+        await tester.pump(); // [MenuItemButton.onPressed] called in a post-frame callback: flutter/flutter@e4a39fa2e
+      }
+
+      testWidgets('copies channel link to clipboard', (tester) async {
+        await prepare();
+        final narrow = ChannelNarrow(someChannel.streamId);
+        await showFromAppBar(tester, narrow: narrow);
+
+        await tapCopyChannelLinkButton(tester);
+        await tester.pump(Duration.zero);
+        final expectedLink = narrowLink(store, narrow).toString();
+        check(await Clipboard.getData('text/plain')).isNotNull().text.equals(expectedLink);
+      });
+    });
   });
 
   group('topic action sheet', () {
