share: Support switching accounts

rajveermalviya · chrisbobbe · rajveermalviya · commit 955d505be2f1 · 2025-12-12T12:18:50.000+05:30
Co-Authored-By: Chris Bobbe <cbobbe@zulip.com> Fixes: #1779
diff --git a/assets/l10n/app_en.arb b/assets/l10n/app_en.arb
@@ -1216,6 +1216,10 @@
       "allChannelsPageTitle": {"type": "String", "example": "All channels"}
     }
   },
+  "shareChooseAccountLabel": "Choose an account",
+  "@shareChooseAccountLabel": {
+    "description": "Label for the page about selecting an account to share content received from other apps."
+  },
   "mainMenuMyProfile": "My profile",
   "@mainMenuMyProfile": {
     "description": "Label for main-menu button leading to the user's own profile."
diff --git a/lib/generated/l10n/zulip_localizations.dart b/lib/generated/l10n/zulip_localizations.dart
@@ -1779,6 +1779,12 @@ abstract class ZulipLocalizations {
   /// **'Try going to <z-link>{allChannelsPageTitle}</z-link> and joining some of them.'**
   String channelsEmptyPlaceholderMessage(String allChannelsPageTitle);
 
+  /// Label for the page about selecting an account to share content received from other apps.
+  ///
+  /// In en, this message translates to:
+  /// **'Choose an account'**
+  String get shareChooseAccountLabel;
+
   /// Label for main-menu button leading to the user's own profile.
   ///
   /// In en, this message translates to:
diff --git a/lib/generated/l10n/zulip_localizations_ar.dart b/lib/generated/l10n/zulip_localizations_ar.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/generated/l10n/zulip_localizations_de.dart b/lib/generated/l10n/zulip_localizations_de.dart
@@ -1041,6 +1041,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Mein Profil';
 
diff --git a/lib/generated/l10n/zulip_localizations_el.dart b/lib/generated/l10n/zulip_localizations_el.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsEl extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/generated/l10n/zulip_localizations_en.dart b/lib/generated/l10n/zulip_localizations_en.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/generated/l10n/zulip_localizations_es.dart b/lib/generated/l10n/zulip_localizations_es.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsEs extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/generated/l10n/zulip_localizations_fr.dart b/lib/generated/l10n/zulip_localizations_fr.dart
@@ -1038,6 +1038,9 @@ class ZulipLocalizationsFr extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Mon profil';
 
diff --git a/lib/generated/l10n/zulip_localizations_he.dart b/lib/generated/l10n/zulip_localizations_he.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsHe extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/generated/l10n/zulip_localizations_hu.dart b/lib/generated/l10n/zulip_localizations_hu.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsHu extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/generated/l10n/zulip_localizations_it.dart b/lib/generated/l10n/zulip_localizations_it.dart
@@ -1035,6 +1035,9 @@ class ZulipLocalizationsIt extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Il mio profilo';
 
diff --git a/lib/generated/l10n/zulip_localizations_ja.dart b/lib/generated/l10n/zulip_localizations_ja.dart
@@ -997,6 +997,9 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => '自分のプロフィール';
 
diff --git a/lib/generated/l10n/zulip_localizations_nb.dart b/lib/generated/l10n/zulip_localizations_nb.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/generated/l10n/zulip_localizations_pl.dart b/lib/generated/l10n/zulip_localizations_pl.dart
@@ -1038,6 +1038,9 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
     return 'Spróbuj skorzystać z <z-link>$allChannelsPageTitle</z-link> i dołączyć do nich.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Mój profil';
 
diff --git a/lib/generated/l10n/zulip_localizations_ru.dart b/lib/generated/l10n/zulip_localizations_ru.dart
@@ -1049,6 +1049,9 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
     return 'Вы можете просмотреть <z-link>$allChannelsPageTitle</z-link> и присоединиться к некоторым из них.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Мой профиль';
 
diff --git a/lib/generated/l10n/zulip_localizations_sk.dart b/lib/generated/l10n/zulip_localizations_sk.dart
@@ -1024,6 +1024,9 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Môj profil';
 
diff --git a/lib/generated/l10n/zulip_localizations_sl.dart b/lib/generated/l10n/zulip_localizations_sl.dart
@@ -1057,6 +1057,9 @@ class ZulipLocalizationsSl extends ZulipLocalizations {
     return 'Poskusite odpreti <z-link>$allChannelsPageTitle</z-link> in se jim pridružiti.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Moj profil';
 
diff --git a/lib/generated/l10n/zulip_localizations_uk.dart b/lib/generated/l10n/zulip_localizations_uk.dart
@@ -1039,6 +1039,9 @@ class ZulipLocalizationsUk extends ZulipLocalizations {
     return 'Спробуйте перейти за посиланням <z-link>$allChannelsPageTitle</z-link> та приєднатися до деяких із них.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'Мій профіль';
 
diff --git a/lib/generated/l10n/zulip_localizations_zh.dart b/lib/generated/l10n/zulip_localizations_zh.dart
@@ -1022,6 +1022,9 @@ class ZulipLocalizationsZh extends ZulipLocalizations {
     return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';
   }
 
+  @override
+  String get shareChooseAccountLabel => 'Choose an account';
+
   @override
   String get mainMenuMyProfile => 'My profile';
 
diff --git a/lib/widgets/share.dart b/lib/widgets/share.dart
@@ -5,14 +5,17 @@ import 'package:flutter/material.dart';
 import 'package:flutter/scheduler.dart';
 import 'package:mime/mime.dart';
 
+import '../api/core.dart';
 import '../generated/l10n/zulip_localizations.dart';
 import '../host/android_intents.dart';
 import '../log.dart';
 import '../model/binding.dart';
 import '../model/narrow.dart';
+import 'action_sheet.dart';
 import 'app.dart';
 import 'compose_box.dart';
 import 'dialog.dart';
+import 'home.dart';
 import 'icons.dart';
 import 'image.dart';
 import 'message_list.dart';
@@ -197,10 +200,13 @@ class ShareSheet extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
+    final globalStore = GlobalStoreWidget.of(context);
     final store = PerAccountStoreWidget.of(context);
     final designVariables = DesignVariables.of(context);
     final zulipLocalizations = ZulipLocalizations.of(context);
 
+    final hasMultipleAccounts = globalStore.accountIds.length > 1;
+
     Widget mkTabLabel({required String text, required IconData icon}) {
       return ConstrainedBox(
         constraints: BoxConstraints(minHeight: 42),
@@ -225,15 +231,26 @@ class ShareSheet extends StatelessWidget {
       length: 2,
       child: Column(children: [
         Row(children: [
-          Padding(
-            padding: EdgeInsets.symmetric(vertical: 7, horizontal: 11),
-            child: AvatarShape(
-              size: 28,
-              borderRadius: 4,
-              child: RealmContentNetworkImage(
-                store.resolvedRealmIcon,
-                filterQuality: FilterQuality.medium,
-                fit: BoxFit.cover))),
+          GestureDetector(
+            behavior: HitTestBehavior.opaque,
+            onTap: !hasMultipleAccounts
+              ? null
+              : () {
+                  ChooseAccountForShareDialog.show(
+                    pageContext: context,
+                    selectedAccountId: store.accountId,
+                    sharedFiles: sharedFiles,
+                    sharedText: sharedText);
+                },
+            child: Padding(
+              padding: EdgeInsets.symmetric(vertical: 7, horizontal: 11),
+              child: AvatarShape(
+                size: 28,
+                borderRadius: 4,
+                child: RealmContentNetworkImage(
+                  store.resolvedRealmIcon,
+                  filterQuality: FilterQuality.medium,
+                  fit: BoxFit.cover)))),
           Expanded(child: TabBar(
             padding: EdgeInsets.zero,
             labelPadding: EdgeInsets.symmetric(horizontal: 4),
@@ -284,3 +301,107 @@ class ShareSheet extends StatelessWidget {
       ]));
   }
 }
+
+class ChooseAccountForShareDialog extends StatefulWidget {
+  const ChooseAccountForShareDialog({
+    super.key,
+    required this.sharedFiles,
+    required this.sharedText,
+  });
+
+  final Iterable<FileToUpload>? sharedFiles;
+  final String? sharedText;
+
+  static void show({
+    required BuildContext pageContext,
+    required int selectedAccountId,
+    required Iterable<FileToUpload>? sharedFiles,
+    required String? sharedText,
+  }) async {
+    unawaited(showModalBottomSheet<void>(
+      context: pageContext,
+      // Clip.hardEdge looks bad; Clip.antiAliasWithSaveLayer looks pixel-perfect
+      // on my iPhone 13 Pro but is marked as "much slower":
+      //   https://api.flutter.dev/flutter/dart-ui/Clip.html
+      clipBehavior: Clip.antiAlias,
+      useSafeArea: true,
+      isScrollControlled: true,
+      builder: (_) {
+        return SafeArea(
+          minimum: const EdgeInsets.only(bottom: 16),
+          child: ChooseAccountForShareDialog(
+            sharedFiles: sharedFiles,
+            sharedText: sharedText));
+      }));
+  }
+
+  @override
+  State<ChooseAccountForShareDialog> createState() => _ChooseAccountForShareDialogState();
+}
+
+class _ChooseAccountForShareDialogState extends State<ChooseAccountForShareDialog> {
+  bool _hasUpdatedAccountsOnce = false;
+
+  @override
+  void didChangeDependencies() {
+    super.didChangeDependencies();
+
+    final globalStore = GlobalStoreWidget.of(context);
+
+    if (_hasUpdatedAccountsOnce) return;
+    _hasUpdatedAccountsOnce = true;
+
+    globalStore.ensureFreshRealmMetadata();
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    final globalStore = GlobalStoreWidget.of(context);
+    final zulipLocalizations = ZulipLocalizations.of(context);
+
+    final accountIds = List<int>.unmodifiable(globalStore.accountIds);
+
+    // TODO(#1038) align the design of this dialog to other
+    //   choose account dialogs
+    final content = SliverList.builder(
+      itemCount: accountIds.length,
+      itemBuilder: (context, index) {
+        final accountId = accountIds[index];
+        final account = globalStore.getAccount(accountId);
+        if (account == null) return const SizedBox.shrink();
+
+        final resolvedRealmIconUrl =
+          account.realmUrl.resolveUri(account.realmIcon!);
+
+        return ListTile(
+          onTap: () {
+            // First change home page account to the selected account.
+            HomePage.navigate(context, accountId: accountId);
+            // Then push a new share dialog for the selected account.
+            ShareSheet.show(
+              pageContext: context,
+              initialAccountId: accountId,
+              sharedFiles: widget.sharedFiles,
+              sharedText: widget.sharedText);
+          },
+          splashColor: Colors.transparent,
+          leading: AspectRatio(
+            aspectRatio: 1,
+            child: account.realmIcon == null
+              ? null
+              : Image.network(
+                  resolvedRealmIconUrl.toString(),
+                  headers: userAgentHeader(),
+                  filterQuality: FilterQuality.medium,
+                  fit: BoxFit.cover)),
+          title: Text(account.realmName ?? account.realmUrl.toString()),
+          subtitle: Text(account.email));
+      });
+
+    return DraggableScrollableModalBottomSheet(
+      header: Padding(
+        padding: const EdgeInsets.only(top: 8),
+        child: BottomSheetHeader(title: zulipLocalizations.shareChooseAccountLabel)),
+      contentSliver: content);
+  }
+}

Original file line number	Diff line number	Diff line change
`@@ -1022,6 +1022,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {`
`1022`	`1022`	`return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';`
`1023`	`1023`	`}`
`1024`	`1024`
	`1025`	`+ @override`
	`1026`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`1027`	`+`
`1025`	`1028`	`@override`
`1026`	`1029`	`String get mainMenuMyProfile => 'My profile';`
`1027`	`1030`
Original file line number	Diff line number	Diff line change
`@@ -1041,6 +1041,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {`
`1041`	`1041`	`return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';`
`1042`	`1042`	`}`
`1043`	`1043`
	`1044`	`+ @override`
	`1045`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`1046`	`+`
`1044`	`1047`	`@override`
`1045`	`1048`	`String get mainMenuMyProfile => 'Mein Profil';`
`1046`	`1049`
Original file line number	Diff line number	Diff line change
`@@ -1038,6 +1038,9 @@ class ZulipLocalizationsFr extends ZulipLocalizations {`
`1038`	`1038`	`return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';`
`1039`	`1039`	`}`
`1040`	`1040`
	`1041`	`+ @override`
	`1042`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`1043`	`+`
`1041`	`1044`	`@override`
`1042`	`1045`	`String get mainMenuMyProfile => 'Mon profil';`
`1043`	`1046`