emoji_reaction: Add labeled Semantics container node for reactions list

chrisbobbe · chrisbobbe · commit 41e3d57f2cde · 2025-08-01T11:55:09.000-07:00
diff --git a/assets/l10n/app_en.arb b/assets/l10n/app_en.arb
@@ -973,6 +973,10 @@
   "@reactedEmojiSelfUser": {
     "description": "Display name for the user themself, to show on an emoji reaction added by the user."
   },
+  "reactionChipsLabel": "Reactions",
+  "@reactionChipsLabel": {
+    "description": "Text identifying the container of reaction chips on a message. (An accessibility label for assistive technology.)"
+  },
   "reactionChipLabel": "{emojiName}: {votes}",
   "@reactionChipLabel": {
     "description": "Text describing a reaction chip, with the emoji name and a list or number of votes. (An accessibility label for assistive technology.)",
diff --git a/lib/generated/l10n/zulip_localizations.dart b/lib/generated/l10n/zulip_localizations.dart
@@ -1469,6 +1469,12 @@ abstract class ZulipLocalizations {
   /// **'You'**
   String get reactedEmojiSelfUser;
 
+  /// Text identifying the container of reaction chips on a message. (An accessibility label for assistive technology.)
+  ///
+  /// In en, this message translates to:
+  /// **'Reactions'**
+  String get reactionChipsLabel;
+
   /// Text describing a reaction chip, with the emoji name and a list or number of votes. (An accessibility label for assistive technology.)
   ///
   /// In en, this message translates to:
diff --git a/lib/generated/l10n/zulip_localizations_ar.dart b/lib/generated/l10n/zulip_localizations_ar.dart
@@ -796,6 +796,9 @@ class ZulipLocalizationsAr extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'You';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_de.dart b/lib/generated/l10n/zulip_localizations_de.dart
@@ -816,6 +816,9 @@ class ZulipLocalizationsDe extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'Du';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_en.dart b/lib/generated/l10n/zulip_localizations_en.dart
@@ -796,6 +796,9 @@ class ZulipLocalizationsEn extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'You';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_fr.dart b/lib/generated/l10n/zulip_localizations_fr.dart
@@ -796,6 +796,9 @@ class ZulipLocalizationsFr extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'You';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_it.dart b/lib/generated/l10n/zulip_localizations_it.dart
@@ -811,6 +811,9 @@ class ZulipLocalizationsIt extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'Tu';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_ja.dart b/lib/generated/l10n/zulip_localizations_ja.dart
@@ -793,6 +793,9 @@ class ZulipLocalizationsJa extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'You';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_nb.dart b/lib/generated/l10n/zulip_localizations_nb.dart
@@ -796,6 +796,9 @@ class ZulipLocalizationsNb extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'You';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_pl.dart b/lib/generated/l10n/zulip_localizations_pl.dart
@@ -806,6 +806,9 @@ class ZulipLocalizationsPl extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'Ty';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_ru.dart b/lib/generated/l10n/zulip_localizations_ru.dart
@@ -810,6 +810,9 @@ class ZulipLocalizationsRu extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'Вы';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_sk.dart b/lib/generated/l10n/zulip_localizations_sk.dart
@@ -798,6 +798,9 @@ class ZulipLocalizationsSk extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'You';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_sl.dart b/lib/generated/l10n/zulip_localizations_sl.dart
@@ -821,6 +821,9 @@ class ZulipLocalizationsSl extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'Vi';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_uk.dart b/lib/generated/l10n/zulip_localizations_uk.dart
@@ -809,6 +809,9 @@ class ZulipLocalizationsUk extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'Ви';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/generated/l10n/zulip_localizations_zh.dart b/lib/generated/l10n/zulip_localizations_zh.dart
@@ -796,6 +796,9 @@ class ZulipLocalizationsZh extends ZulipLocalizations {
   @override
   String get reactedEmojiSelfUser => 'You';
 
+  @override
+  String get reactionChipsLabel => 'Reactions';
+
   @override
   String reactionChipLabel(String emojiName, String votes) {
     return '$emojiName: $votes';
diff --git a/lib/widgets/emoji_reaction.dart b/lib/widgets/emoji_reaction.dart
@@ -122,15 +122,22 @@ class ReactionChipsList extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
+    final zulipLocalizations = ZulipLocalizations.of(context);
     final store = PerAccountStoreWidget.of(context);
     final displayEmojiReactionUsers = store.userSettings.displayEmojiReactionUsers ?? false;
     final showNames = displayEmojiReactionUsers && reactions.total <= 3;
 
-    return Wrap(spacing: 4, runSpacing: 4, crossAxisAlignment: WrapCrossAlignment.center,
+    Widget result = Wrap(spacing: 4, runSpacing: 4, crossAxisAlignment: WrapCrossAlignment.center,
       children: reactions.aggregated.map((reactionVotes) => ReactionChip(
         showName: showNames,
         messageId: messageId, reactionWithVotes: reactionVotes),
       ).toList());
+
+    return Semantics(
+      label: zulipLocalizations.reactionChipsLabel,
+      container: true,
+      explicitChildNodes: true,
+      child: result);
   }
 }
 
