share: Support switching accounts

rajveermalviya · chrisbobbe · rajveermalviya · commit 6fd5a6b3fa73 · 2025-12-30T23:16:06.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
@@ -1220,6 +1220,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
@@ -1787,6 +1787,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
@@ -1025,6 +1025,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
@@ -1050,6 +1050,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {
     return 'Versuche, nach <z-link>$allChannelsPageTitle</z-link> zu gehen und einigen davon beizutreten.';
   }
 
+  @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
@@ -1025,6 +1025,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
@@ -1025,6 +1025,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
@@ -1025,6 +1025,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
@@ -1041,6 +1041,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
@@ -1025,6 +1025,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
@@ -1025,6 +1025,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
@@ -1038,6 +1038,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
@@ -1000,6 +1000,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
@@ -1025,6 +1025,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
@@ -1042,6 +1042,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
@@ -1052,6 +1052,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
@@ -1027,6 +1027,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
@@ -1060,6 +1060,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
@@ -1043,6 +1043,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_vi.dart b/lib/generated/l10n/zulip_localizations_vi.dart
@@ -1025,6 +1025,9 @@ class ZulipLocalizationsVi 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_zh.dart b/lib/generated/l10n/zulip_localizations_zh.dart
@@ -1025,6 +1025,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,18 @@ 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 '../model/store.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';
@@ -196,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),
@@ -224,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),
@@ -283,3 +301,111 @@ 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.refreshRealmMetadata();
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    final globalStore = GlobalStoreWidget.of(context);
+    final zulipLocalizations = ZulipLocalizations.of(context);
+
+    final accounts = List<Account>.unmodifiable(globalStore.accounts);
+
+    // TODO(#1038) align the design of this dialog to other
+    //   choose account dialogs
+    final content = SliverList.builder(
+      itemCount: accounts.length,
+      itemBuilder: (context, index) {
+        final account = accounts[index];
+        final accountId = account.id;
+
+        final resolvedRealmIconUrl =
+          account.realmIcon == null
+            ? null
+            : account.realmUrl.resolveUri(account.realmIcon!);
+
+        return ListTile(
+          onTap: () {
+            if (globalStore.getAccount(accountId) == null) return;
+
+            // 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: AvatarShape(
+            size: 56,
+            borderRadius: 4,
+            child: resolvedRealmIconUrl == null
+              ? const SizedBox.shrink()
+              : 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: BottomSheetHeader(
+        title: zulipLocalizations.shareChooseAccountLabel,
+        outerVerticalPadding: true),
+      contentSliver: content);
+  }
+}

Original file line number	Diff line number	Diff line change
`@@ -1025,6 +1025,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {`
`1025`	`1025`	`return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';`
`1026`	`1026`	`}`
`1027`	`1027`
	`1028`	`+ @override`
	`1029`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`1030`	`+`
`1028`	`1031`	`@override`
`1029`	`1032`	`String get mainMenuMyProfile => 'My profile';`
`1030`	`1033`
Original file line number	Diff line number	Diff line change
`@@ -1050,6 +1050,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {`
`1050`	`1050`	`return 'Versuche, nach <z-link>$allChannelsPageTitle</z-link> zu gehen und einigen davon beizutreten.';`
`1051`	`1051`	`}`
`1052`	`1052`
	`1053`	`+ @override`
	`1054`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`1055`	`+`
`1053`	`1056`	`@override`
`1054`	`1057`	`String get mainMenuMyProfile => 'Mein Profil';`
`1055`	`1058`
Original file line number	Diff line number	Diff line change
`@@ -1041,6 +1041,9 @@ class ZulipLocalizationsFr 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 => 'Mon profil';`
`1046`	`1049`