reactions: Add sheet to view who reacted to a message

chimnayajith · chimnayajith · commit 6bfe471f9f8e · 2025-01-16T11:01:59.000+05:30
Fixes #740
diff --git a/assets/l10n/app_en.arb b/assets/l10n/app_en.arb
@@ -104,6 +104,10 @@
   "@actionSheetOptionShare":  {
     "description": "Label for share button on action sheet."
   },
+  "actionSheetOptionViewReactions": "View Reactions",
+  "@actionSheetOptionViewReactions":  {
+    "description": "Label for View Reactions button on action sheet."
+  },
   "actionSheetOptionQuoteAndReply": "Quote and reply",
   "@actionSheetOptionQuoteAndReply":  {
     "description": "Label for Quote and reply button on action sheet."
diff --git a/lib/generated/l10n/zulip_localizations.dart b/lib/generated/l10n/zulip_localizations.dart
@@ -261,6 +261,12 @@ abstract class ZulipLocalizations {
   /// **'Share'**
   String get actionSheetOptionShare;
 
+  /// Label for View Reactions button on action sheet.
+  ///
+  /// In en, this message translates to:
+  /// **'View Reactions'**
+  String get actionSheetOptionViewReactions;
+
   /// Label for Quote and reply button on action sheet.
   ///
   /// In en, this message translates to:
diff --git a/lib/generated/l10n/zulip_localizations_ar.dart b/lib/generated/l10n/zulip_localizations_ar.dart
@@ -88,6 +88,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
   @override
   String get actionSheetOptionShare => 'Share';
 
+  @override
+  String get actionSheetOptionViewReactions => 'View Reactions';
+
   @override
   String get actionSheetOptionQuoteAndReply => 'Quote and reply';
 
diff --git a/lib/generated/l10n/zulip_localizations_en.dart b/lib/generated/l10n/zulip_localizations_en.dart
@@ -88,6 +88,9 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
   @override
   String get actionSheetOptionShare => 'Share';
 
+  @override
+  String get actionSheetOptionViewReactions => 'View Reactions';
+
   @override
   String get actionSheetOptionQuoteAndReply => 'Quote and reply';
 
diff --git a/lib/generated/l10n/zulip_localizations_ja.dart b/lib/generated/l10n/zulip_localizations_ja.dart
@@ -88,6 +88,9 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
   @override
   String get actionSheetOptionShare => 'Share';
 
+  @override
+  String get actionSheetOptionViewReactions => 'View Reactions';
+
   @override
   String get actionSheetOptionQuoteAndReply => 'Quote and reply';
 
diff --git a/lib/generated/l10n/zulip_localizations_nb.dart b/lib/generated/l10n/zulip_localizations_nb.dart
@@ -88,6 +88,9 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
   @override
   String get actionSheetOptionShare => 'Share';
 
+  @override
+  String get actionSheetOptionViewReactions => 'View Reactions';
+
   @override
   String get actionSheetOptionQuoteAndReply => 'Quote and reply';
 
diff --git a/lib/generated/l10n/zulip_localizations_pl.dart b/lib/generated/l10n/zulip_localizations_pl.dart
@@ -88,6 +88,9 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
   @override
   String get actionSheetOptionShare => 'Udostępnij';
 
+  @override
+  String get actionSheetOptionViewReactions => 'View Reactions';
+
   @override
   String get actionSheetOptionQuoteAndReply => 'Odpowiedz cytując';
 
diff --git a/lib/generated/l10n/zulip_localizations_ru.dart b/lib/generated/l10n/zulip_localizations_ru.dart
@@ -88,6 +88,9 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
   @override
   String get actionSheetOptionShare => 'Поделиться';
 
+  @override
+  String get actionSheetOptionViewReactions => 'View Reactions';
+
   @override
   String get actionSheetOptionQuoteAndReply => 'Ответить с цитированием';
 
diff --git a/lib/generated/l10n/zulip_localizations_sk.dart b/lib/generated/l10n/zulip_localizations_sk.dart
@@ -88,6 +88,9 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
   @override
   String get actionSheetOptionShare => 'Zdielať';
 
+  @override
+  String get actionSheetOptionViewReactions => 'View Reactions';
+
   @override
   String get actionSheetOptionQuoteAndReply => 'Citovať a odpovedať';
 
diff --git a/lib/widgets/action_sheet.dart b/lib/widgets/action_sheet.dart
@@ -378,6 +378,8 @@ void showMessageActionSheet({required BuildContext context, required Message mes
 
   final optionButtons = [
     ReactionButtons(message: message, pageContext: context),
+    if((message.reactions?.total ?? 0) > 0)
+      ViewReactionsButton(message: message, pageContext: context),
     StarButton(message: message, pageContext: context),
     if (isComposeBoxOffered)
       QuoteAndReplyButton(message: message, pageContext: context),
@@ -678,6 +680,24 @@ class QuoteAndReplyButton extends MessageActionSheetMenuItemButton {
   }
 }
 
+class ViewReactionsButton extends MessageActionSheetMenuItemButton {
+  ViewReactionsButton({super.key, required super.message, required super.pageContext});
+
+  @override IconData get icon => ZulipIcons.reactions;
+
+  @override
+  String label(ZulipLocalizations zulipLocalizations) {
+    return zulipLocalizations.actionSheetOptionViewReactions;
+  }
+
+  @override void onPressed() async {
+    showReactionListSheet(
+      pageContext,
+      reactionList: message.reactions?.aggregated,
+    );
+  }
+}
+
 class MarkAsUnreadButton extends MessageActionSheetMenuItemButton {
   MarkAsUnreadButton({super.key, required super.message, required super.pageContext});
 
diff --git a/lib/widgets/emoji_reaction.dart b/lib/widgets/emoji_reaction.dart
@@ -6,10 +6,13 @@ import '../api/route/messages.dart';
 import '../generated/l10n/zulip_localizations.dart';
 import '../model/autocomplete.dart';
 import '../model/emoji.dart';
+import '../model/store.dart';
 import 'color.dart';
+import 'content.dart';
 import 'dialog.dart';
 import 'emoji.dart';
 import 'inset_shadow.dart';
+import 'profile.dart';
 import 'store.dart';
 import 'text.dart';
 import 'theme.dart';
@@ -127,23 +130,36 @@ class ReactionChipsList extends StatelessWidget {
     final showNames = displayEmojiReactionUsers && reactions.total <= 3;
 
     return Wrap(spacing: 4, runSpacing: 4, crossAxisAlignment: WrapCrossAlignment.center,
-      children: reactions.aggregated.map((reactionVotes) => ReactionChip(
+      children: reactions.aggregated.map((reactionVotes) {
+        final index = reactions.aggregated.indexOf(reactionVotes);
+        return ReactionChip(
         showName: showNames,
-        messageId: messageId, reactionWithVotes: reactionVotes),
-      ).toList());
+        messageId: messageId,
+        reactionWithVotes: reactionVotes,
+        onLongPress:(context){
+          showReactionListSheet(
+            context,
+            reactionList: reactions.aggregated,
+            initialTabIndex: index,
+            );
+        }
+      );
+    }).toList());
   }
 }
 
 class ReactionChip extends StatelessWidget {
   final bool showName;
   final int messageId;
   final ReactionWithVotes reactionWithVotes;
+  final void Function(BuildContext context)? onLongPress;
 
   const ReactionChip({
     super.key,
     required this.showName,
     required this.messageId,
     required this.reactionWithVotes,
+    this.onLongPress,
   });
 
   @override
@@ -206,6 +222,11 @@ class ReactionChip extends StatelessWidget {
           customBorder: shape,
           splashColor: splashColor,
           highlightColor: highlightColor,
+          onLongPress: (){
+             if (onLongPress != null) {
+                onLongPress!(context);
+              }
+          },
           onTap: () {
             (selfVoted ? removeReaction : addReaction).call(store.connection,
               messageId: messageId,
@@ -266,6 +287,203 @@ class ReactionChip extends StatelessWidget {
   }
 }
 
+void showReactionListSheet(
+  BuildContext context, {
+  required List<ReactionWithVotes>? reactionList,
+  int initialTabIndex = 0,
+}) {
+  final store = PerAccountStoreWidget.of(context);
+
+  if (reactionList == null || reactionList.isEmpty) return;
+
+  showModalBottomSheet<void>(
+    context: context,
+    clipBehavior: Clip.antiAlias,
+    useSafeArea: true,
+    isScrollControlled: true,
+    builder: (BuildContext modalContext) {
+      return ConstrainedBox(
+        constraints: BoxConstraints(
+          maxHeight: MediaQuery.of(context).size.height * 0.7,
+        ),
+        child: SafeArea(
+          minimum: const EdgeInsets.only(bottom: 16),
+          child: Padding(
+            padding: const EdgeInsets.fromLTRB(16, 0, 16, 0),
+            child: Column(
+              crossAxisAlignment: CrossAxisAlignment.stretch,
+              mainAxisSize: MainAxisSize.min,
+              children: [
+                Flexible(
+                  child: InsetShadowBox(
+                    top: 8,
+                    bottom: 8,
+                    color: DesignVariables.of(context).bgContextMenu,
+                    child: PerAccountStoreWidget(
+                      accountId: store.accountId,
+                      child: ReactionListContent(
+                        store: store,
+                        reactionList: reactionList,
+                        initialTabIndex: initialTabIndex
+                      ),
+                    ),
+                  ),
+                ),
+                const ReactionSheetCloseButton(),
+              ],
+            ),
+          ),
+        ),
+      );
+    },
+  );
+}
+class ReactionListContent extends StatelessWidget {
+  final PerAccountStore store;
+  final List<ReactionWithVotes> reactionList;
+  final int initialTabIndex;
+
+  const ReactionListContent({
+    super.key,
+    required this.store,
+    required this.reactionList,
+    this.initialTabIndex = 0,
+  });
+
+  @override
+  Widget build(BuildContext context) {
+    final designVariables = DesignVariables.of(context);
+
+    final tabs = reactionList.map((reaction) {
+      final emojiDisplay = store.emojiDisplayFor(
+        emojiType: reaction.reactionType,
+        emojiCode: reaction.emojiCode,
+        emojiName: reaction.emojiName,
+      ).resolve(store.userSettings);
+
+      final emoji = switch (emojiDisplay) {
+        UnicodeEmojiDisplay() => _UnicodeEmoji(emojiDisplay: emojiDisplay),
+        ImageEmojiDisplay() => _ImageEmoji(
+          emojiDisplay: emojiDisplay,
+          emojiName: reaction.emojiName,
+          selected: reaction.userIds.contains(store.selfUserId),
+        ),
+        TextEmojiDisplay() => _TextEmoji(
+          emojiDisplay: emojiDisplay,
+          selected: reaction.userIds.contains(store.selfUserId),
+        ),
+      };
+
+      return Tab(
+        child: Column(
+          mainAxisSize: MainAxisSize.min,
+          mainAxisAlignment: MainAxisAlignment.center,
+          children: [
+            emoji,
+            const SizedBox(height: 4),
+            Text(
+              '${reaction.userIds.length}',
+              style: const TextStyle()
+                .merge(weightVariableTextStyle(context, wght: 600)),
+            ),
+          ],
+        ),
+      );
+    }).toList();
+
+    final tabViews = reactionList.map((reaction) {
+      return ListView.builder(
+        padding: EdgeInsets.zero,
+        itemCount: reaction.userIds.length,
+        itemBuilder: (context, index) {
+          final userId = reaction.userIds.elementAt(index);
+          return ListTile(
+              leading: Avatar(userId: userId, size: 32.0, borderRadius: 3),
+              title: Text(
+                userId == store.selfUserId
+                    ? 'You'
+                    : store.users[userId]?.fullName ?? '(unknown user)',
+                style: TextStyle(
+                  color: designVariables.foreground.withFadedAlpha(0.80),
+                  fontSize: 17,
+                ).merge(weightVariableTextStyle(context, wght: 500)),
+              ),
+              onTap: () {
+                Navigator.push(
+                  context,
+                  ProfilePage.buildRoute(context: context, userId: userId),
+                );
+              },
+            );
+        },
+      );
+    }).toList();
+
+    return DefaultTabController(
+      length: tabs.length,
+      initialIndex: initialTabIndex,
+      child: Column(
+        mainAxisSize: MainAxisSize.min,
+        children: [
+          Padding(
+            padding: const EdgeInsets.only(top: 16.0),
+            child: TabBar(
+              isScrollable: true,
+              tabAlignment: TabAlignment.start,
+              dividerColor: Colors.transparent,
+              indicator: BoxDecoration(
+                color: designVariables.background,
+                borderRadius: BorderRadius.circular(10),
+                border: Border.all(
+                  color: designVariables.foreground.withFadedAlpha(0.2),
+                  width:1
+                )
+              ),
+              splashFactory: NoSplash.splashFactory,
+              indicatorSize: TabBarIndicatorSize.tab,
+              labelColor: designVariables.foreground,
+              unselectedLabelColor: designVariables.foreground,
+              labelStyle: const TextStyle(fontSize: 14)
+                  .merge(weightVariableTextStyle(context, wght: 400)),
+              unselectedLabelStyle: const TextStyle(fontSize: 14)
+                  .merge(weightVariableTextStyle(context, wght: 400)),
+              tabs: tabs,
+            ),
+          ),
+          const SizedBox(height: 8),
+          Flexible(
+            child: TabBarView(children: tabViews),
+          ),
+        ],
+      ),
+    );
+  }
+}
+class ReactionSheetCloseButton extends StatelessWidget {
+  const ReactionSheetCloseButton({super.key});
+
+  @override
+  Widget build(BuildContext context) {
+    final designVariables = DesignVariables.of(context);
+    return TextButton(
+      style: TextButton.styleFrom(
+        minimumSize: const Size.fromHeight(44),
+        padding: const EdgeInsets.all(10),
+        foregroundColor: designVariables.contextMenuCancelText,
+        shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(7)),
+        splashFactory: NoSplash.splashFactory,
+      ).copyWith(backgroundColor: WidgetStateColor.fromMap({
+        WidgetState.pressed: designVariables.contextMenuCancelPressedBg,
+        ~WidgetState.pressed: designVariables.contextMenuCancelBg,
+      })),
+      onPressed: () {
+        Navigator.pop(context);
+      },
+      child: Text(ZulipLocalizations.of(context).dialogClose,
+        style: const TextStyle(fontSize: 20, height: 24 / 20)
+          .merge(weightVariableTextStyle(context, wght: 600))));
+  }
+}
 /// The size of a square emoji (Unicode or image).
 ///
 /// Should be scaled by [_emojiTextScalerClamped].
diff --git a/test/widgets/action_sheet_test.dart b/test/widgets/action_sheet_test.dart
