share: Support switching accounts

rajveermalviya · chrisbobbe · rajveermalviya · commit db3de4bcd78d · 2025-12-10T23:56:24.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
@@ -1147,6 +1147,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
@@ -1689,6 +1689,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
@@ -960,6 +960,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
@@ -979,6 +979,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
@@ -960,6 +960,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
@@ -960,6 +960,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
@@ -960,6 +960,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
@@ -976,6 +976,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
@@ -960,6 +960,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
@@ -960,6 +960,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
@@ -973,6 +973,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
@@ -935,6 +935,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
@@ -960,6 +960,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
@@ -976,6 +976,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
@@ -987,6 +987,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
@@ -962,6 +962,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
@@ -995,6 +995,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
@@ -977,6 +977,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
@@ -960,6 +960,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 '../api/route/realm.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 +201,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 +232,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 +302,129 @@ 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> {
+  late List<int> accountIds;
+  bool _hasUpdatedAccountsOnce = false;
+
+  @override
+  void didChangeDependencies() {
+    super.didChangeDependencies();
+
+    final globalStore = GlobalStoreWidget.of(context);
+    accountIds = List.unmodifiable(globalStore.accountIds);
+
+    if (_hasUpdatedAccountsOnce) return;
+    _hasUpdatedAccountsOnce = true;
+
+    for (final accountId in accountIds) {
+      final account = globalStore.getAccount(accountId);
+      if (account == null) continue;
+
+      unawaited(() async {
+        final GetServerSettingsResult serverSettings;
+        final connection = globalStore.apiConnection(
+          realmUrl: account.realmUrl,
+          zulipFeatureLevel: null);
+        try {
+          serverSettings = await getServerSettings(connection);
+        } catch (_) {
+          return;
+        } finally {
+          connection.close();
+        }
+
+        if (globalStore.getAccount(accountId) != null) {
+          await globalStore.updateRealmData(
+            accountId,
+            realmName: serverSettings.realmName,
+            realmIcon: serverSettings.realmIcon);
+        }
+      }());
+    }
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    final globalStore = GlobalStoreWidget.of(context);
+    final zulipLocalizations = ZulipLocalizations.of(context);
+
+    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
`@@ -960,6 +960,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {`
`960`	`960`	`return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';`
`961`	`961`	`}`
`962`	`962`
	`963`	`+ @override`
	`964`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`965`	`+`
`963`	`966`	`@override`
`964`	`967`	`String get mainMenuMyProfile => 'My profile';`
`965`	`968`
Original file line number	Diff line number	Diff line change
`@@ -979,6 +979,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {`
`979`	`979`	`return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';`
`980`	`980`	`}`
`981`	`981`
	`982`	`+ @override`
	`983`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`984`	`+`
`982`	`985`	`@override`
`983`	`986`	`String get mainMenuMyProfile => 'Mein Profil';`
`984`	`987`
Original file line number	Diff line number	Diff line change
`@@ -976,6 +976,9 @@ class ZulipLocalizationsFr extends ZulipLocalizations {`
`976`	`976`	`return 'Try going to <z-link>$allChannelsPageTitle</z-link> and joining some of them.';`
`977`	`977`	`}`
`978`	`978`
	`979`	`+ @override`
	`980`	`+ String get shareChooseAccountLabel => 'Choose an account';`
	`981`	`+`
`979`	`982`	`@override`
`980`	`983`	`String get mainMenuMyProfile => 'Mon profil';`
`981`	`984`