feat(ui, localization): add delete poll option functionality

This introduces the ability for users to delete options when creating or editing a poll.

A delete icon is now present on each poll option item. Tapping it shows a confirmation dialog. If confirmed, the option is removed from the list. The list enforces the minimum number of required options.

This change includes:
- A new `PollDeleteOptionDialog` and `showPollDeleteOptionDialog` function.
- An `onRemove` callback to the `_PollOptionItem` widget.
- New theming options for the dialog (`actionDialogTitleStyle`, `actionDialogContentStyle`).
- New translations for the delete confirmation dialog across all supported languages.
- Comprehensive widget and golden tests for the new functionality and dialog.

Author: xsahil03x

packages/stream_chat_flutter/lib/src/localization/translations.dart

@@ -445,6 +445,12 @@ abstract class Translations {
   /// The confirmation text shown when the user tries to end a poll.
   String get endVoteConfirmationText;
 
+  /// The label for "delete poll option"
+  String get deletePollOptionLabel;
+
+  /// The question asked while showing delete poll option dialog
+  String get deletePollOptionQuestion;
+
   /// The label for "Create".
   String get createLabel;
 
@@ -1105,6 +1111,13 @@ Attachment limit exceeded: it's not possible to add more than $limit attachments
   String get endVoteConfirmationText =>
       'Are you sure you want to end the vote?';
 
+  @override
+  String get deletePollOptionLabel => 'Delete option?';
+
+  @override
+  String get deletePollOptionQuestion =>
+      'Are you sure you want to delete this option?';
+
   @override
   String get createLabel => 'Create';
 

packages/stream_chat_flutter/lib/src/misc/separated_reorderable_list_view.dart

@@ -1,4 +1,4 @@
-// ignore_for_file: use_is_even_rather_than_modulo
+// ignore_for_file: parameter_assignments, use_is_even_rather_than_modulo
 import 'dart:math' as math;
 import 'package:flutter/material.dart';
 
@@ -79,7 +79,7 @@ class SeparatedReorderableListView extends ReorderableListView {
                 ? (newIndex + 1) ~/ 2
                 : newIndex ~/ 2;
 
-            onReorder(updatedOldIndex, updatedNewIndex);
+            return onReorder(updatedOldIndex, updatedNewIndex);
           },
         );
 }

packages/stream_chat_flutter/lib/src/poll/creator/poll_option_reorderable_list_view.dart

@@ -3,6 +3,7 @@ import 'dart:ui';
 import 'package:collection/collection.dart';
 import 'package:flutter/material.dart';
 import 'package:stream_chat_flutter/src/misc/separated_reorderable_list_view.dart';
+import 'package:stream_chat_flutter/src/poll/creator/stream_delete_option_dialog.dart';
 import 'package:stream_chat_flutter/stream_chat_flutter.dart';
 
 class _NullConst {
@@ -57,6 +58,7 @@ class PollOptionListItem extends StatelessWidget {
     required this.option,
     this.hintText,
     this.focusNode,
+    this.onRemove,
     this.onChanged,
   });
 
@@ -69,6 +71,9 @@ class PollOptionListItem extends StatelessWidget {
   /// The focus node for the text field.
   final FocusNode? focusNode;
 
+  /// Callback called when the poll option item is removed.
+  final ValueSetter<PollOptionItem>? onRemove;
+
   /// Callback called when the poll option item is changed.
   final ValueSetter<PollOptionItem>? onChanged;
 
@@ -78,13 +83,26 @@ class PollOptionListItem extends StatelessWidget {
     final fillColor = theme.optionsTextFieldFillColor;
     final borderRadius = theme.optionsTextFieldBorderRadius;
 
+    final colorTheme = StreamChatTheme.of(context).colorTheme;
+
     return DecoratedBox(
       decoration: BoxDecoration(
         color: fillColor,
         borderRadius: borderRadius,
       ),
       child: Row(
         children: [
+          Padding(
+            padding: const EdgeInsetsDirectional.all(16),
+            child: MouseRegion(
+              cursor: SystemMouseCursors.grab,
+              child: Icon(
+                size: 24,
+                Icons.drag_handle_rounded,
+                color: colorTheme.textLowEmphasis,
+              ),
+            ),
+          ),
           Expanded(
             child: StreamPollTextField(
               initialValue: option.text,
@@ -95,13 +113,27 @@ class PollOptionListItem extends StatelessWidget {
               errorText: option.error,
               errorStyle: theme.optionsTextFieldErrorStyle,
               focusNode: focusNode,
-              onChanged: (text) => onChanged?.call(option.copyWith(text: text)),
+              contentPadding: const EdgeInsets.symmetric(vertical: 18),
+              onChanged: switch (onChanged) {
+                final onChanged? => (text) {
+                    final updated = option.copyWith(text: text);
+                    return onChanged.call(updated);
+                  },
+                _ => null,
+              },
             ),
           ),
-          const SizedBox(
-            width: 48,
-            height: 48,
-            child: Icon(Icons.drag_handle_rounded),
+          IconButton(
+            iconSize: 24,
+            icon: const StreamSvgIcon(icon: StreamSvgIcons.delete),
+            style: IconButton.styleFrom(
+              foregroundColor: colorTheme.textLowEmphasis,
+            ),
+            onLongPress: () {/* Consume long press */},
+            onPressed: switch (onRemove) {
+              final onRemove? => () => onRemove.call(option),
+              _ => null,
+            },
           ),
         ],
       ),
@@ -217,7 +249,7 @@ class _PollOptionReorderableListViewState
     final newOptions = widget.initialOptions;
 
     final optionItemEquality = ListEquality<PollOptionItem>(
-      EqualityBy((it) => (it.id, it.text)),
+      EqualityBy((it) => it.id),
     );
 
     if (optionItemEquality.equals(currOptions, newOptions) case false) {
@@ -263,6 +295,24 @@ class _PollOptionReorderableListViewState
     return null;
   }
 
+  Future<void> _onOptionRemoved(PollOptionItem option) async {
+    final confirm = await showPollDeleteOptionDialog(context: context);
+    if (confirm == null || !confirm) return;
+
+    setState(() {
+      _options.remove(option.id);
+      _focusNodes.remove(option.id)?.dispose();
+    });
+
+    // Ensure we have at least the minimum number of options
+    _ensureMinimumOptions();
+
+    // Notify the parent widget about the change
+    WidgetsBinding.instance.addPostFrameCallback((_) {
+      widget.onOptionsChanged?.call([..._options.values]);
+    });
+  }
+
   void _onOptionChanged(PollOptionItem option) {
     setState(() {
       // Update the changed option.
@@ -370,6 +420,7 @@ class _PollOptionReorderableListViewState
                 option: option,
                 hintText: widget.itemHintText,
                 focusNode: _focusNodes[option.id],
+                onRemove: _onOptionRemoved,
                 onChanged: _onOptionChanged,
               );
             },

packages/stream_chat_flutter/lib/src/poll/creator/stream_delete_option_dialog.dart

@@ -0,0 +1,68 @@
+import 'package:flutter/material.dart';
+import 'package:stream_chat_flutter/src/theme/poll_creator_theme.dart';
+import 'package:stream_chat_flutter/src/theme/stream_chat_theme.dart';
+import 'package:stream_chat_flutter/src/utils/extensions.dart';
+
+/// {@template showPollDeleteOptionDialog}
+/// Shows a dialog that allows the user to confirm deletion of a poll option.
+/// {@endtemplate}
+Future<bool?> showPollDeleteOptionDialog({
+  required BuildContext context,
+}) {
+  return showDialog<bool?>(
+    context: context,
+    builder: (_) => const PollDeleteOptionDialog(),
+  );
+}
+
+/// {@template pollDeleteOptionDialog}
+/// A dialog that allows the user to confirm deletion of a poll option.
+/// {@endtemplate}
+class PollDeleteOptionDialog extends StatelessWidget {
+  /// {@macro pollDeleteOptionDialog}
+  const PollDeleteOptionDialog({super.key});
+
+  @override
+  Widget build(BuildContext context) {
+    final theme = StreamChatTheme.of(context);
+    final pollCreatorTheme = StreamPollCreatorTheme.of(context);
+
+    final actions = [
+      TextButton(
+        onPressed: () => Navigator.of(context).maybePop(false),
+        style: TextButton.styleFrom(
+          textStyle: theme.textTheme.headlineBold,
+          foregroundColor: theme.colorTheme.accentPrimary,
+          disabledForegroundColor: theme.colorTheme.disabled,
+        ),
+        child: Text(context.translations.cancelLabel.toUpperCase()),
+      ),
+      TextButton(
+        onPressed: () => Navigator.of(context).maybePop(true),
+        style: TextButton.styleFrom(
+          textStyle: theme.textTheme.headlineBold,
+          foregroundColor: theme.colorTheme.accentPrimary,
+          disabledForegroundColor: theme.colorTheme.disabled,
+        ),
+        child: Text(context.translations.deleteLabel.toUpperCase()),
+      ),
+    ];
+
+    return AlertDialog(
+      title: Text(
+        context.translations.deletePollOptionLabel,
+        style: pollCreatorTheme.actionDialogTitleStyle,
+      ),
+      content: Text(
+        context.translations.deletePollOptionQuestion,
+        style: pollCreatorTheme.actionDialogContentStyle,
+      ),
+      actions: actions,
+      titlePadding: const EdgeInsetsDirectional.fromSTEB(16, 24, 16, 4),
+      contentPadding: const EdgeInsetsDirectional.fromSTEB(16, 4, 16, 24),
+      shape: RoundedRectangleBorder(borderRadius: BorderRadius.circular(16)),
+      actionsPadding: const EdgeInsets.all(8),
+      backgroundColor: theme.colorTheme.appBg,
+    );
+  }
+}

packages/stream_chat_flutter/lib/src/theme/poll_creator_theme.dart

@@ -76,6 +76,8 @@ class StreamPollCreatorThemeData with Diagnosticable {
     this.switchListTileTitleStyle,
     this.switchListTileErrorStyle,
     this.switchListTileBorderRadius,
+    this.actionDialogTitleStyle,
+    this.actionDialogContentStyle,
   });
 
   /// The background color of the poll creator.
@@ -133,6 +135,12 @@ class StreamPollCreatorThemeData with Diagnosticable {
   /// The border radius of the switch list tile.
   final BorderRadius? switchListTileBorderRadius;
 
+  /// The text style of the action dialog title.
+  final TextStyle? actionDialogTitleStyle;
+
+  /// The text style of the action dialog content.
+  final TextStyle? actionDialogContentStyle;
+
   /// Copies this [StreamPollCreatorThemeData] with some new values.
   StreamPollCreatorThemeData copyWith({
     Color? backgroundColor,
@@ -153,6 +161,8 @@ class StreamPollCreatorThemeData with Diagnosticable {
     TextStyle? switchListTileTitleStyle,
     TextStyle? switchListTileErrorStyle,
     BorderRadius? switchListTileBorderRadius,
+    TextStyle? actionDialogTitleStyle,
+    TextStyle? actionDialogContentStyle,
   }) {
     return StreamPollCreatorThemeData(
       backgroundColor: backgroundColor ?? this.backgroundColor,
@@ -186,6 +196,10 @@ class StreamPollCreatorThemeData with Diagnosticable {
           switchListTileErrorStyle ?? this.switchListTileErrorStyle,
       switchListTileBorderRadius:
           switchListTileBorderRadius ?? this.switchListTileBorderRadius,
+      actionDialogTitleStyle:
+          actionDialogTitleStyle ?? this.actionDialogTitleStyle,
+      actionDialogContentStyle:
+          actionDialogContentStyle ?? this.actionDialogContentStyle,
     );
   }
 
@@ -224,6 +238,10 @@ class StreamPollCreatorThemeData with Diagnosticable {
           other.switchListTileErrorStyle ?? switchListTileErrorStyle,
       switchListTileBorderRadius:
           other.switchListTileBorderRadius ?? switchListTileBorderRadius,
+      actionDialogTitleStyle:
+          other.actionDialogTitleStyle ?? actionDialogTitleStyle,
+      actionDialogContentStyle:
+          other.actionDialogContentStyle ?? actionDialogContentStyle,
     );
   }
 
@@ -268,6 +286,10 @@ class StreamPollCreatorThemeData with Diagnosticable {
           a.switchListTileErrorStyle, b.switchListTileErrorStyle, t),
       switchListTileBorderRadius: BorderRadius.lerp(
           a.switchListTileBorderRadius, b.switchListTileBorderRadius, t),
+      actionDialogTitleStyle: TextStyle.lerp(
+          a.actionDialogTitleStyle, b.actionDialogTitleStyle, t),
+      actionDialogContentStyle: TextStyle.lerp(
+          a.actionDialogContentStyle, b.actionDialogContentStyle, t),
     );
   }
 
@@ -293,7 +315,9 @@ class StreamPollCreatorThemeData with Diagnosticable {
           other.switchListTileFillColor == switchListTileFillColor &&
           other.switchListTileTitleStyle == switchListTileTitleStyle &&
           other.switchListTileErrorStyle == switchListTileErrorStyle &&
-          other.switchListTileBorderRadius == switchListTileBorderRadius;
+          other.switchListTileBorderRadius == switchListTileBorderRadius &&
+          other.actionDialogTitleStyle == actionDialogTitleStyle &&
+          other.actionDialogContentStyle == actionDialogContentStyle;
 
   @override
   int get hashCode =>
@@ -314,5 +338,7 @@ class StreamPollCreatorThemeData with Diagnosticable {
       switchListTileFillColor.hashCode ^
       switchListTileTitleStyle.hashCode ^
       switchListTileErrorStyle.hashCode ^
-      switchListTileBorderRadius.hashCode;
+      switchListTileBorderRadius.hashCode ^
+      actionDialogTitleStyle.hashCode ^
+      actionDialogContentStyle.hashCode;
 }

packages/stream_chat_flutter/lib/src/theme/stream_chat_theme.dart

@@ -379,6 +379,12 @@ class StreamChatThemeData {
           color: colorTheme.accentError,
         ),
         switchListTileBorderRadius: BorderRadius.circular(12),
+        actionDialogTitleStyle: textTheme.headlineBold.copyWith(
+          color: colorTheme.textHighEmphasis,
+        ),
+        actionDialogContentStyle: textTheme.body.copyWith(
+          color: colorTheme.textHighEmphasis,
+        ),
       ),
       pollInteractorTheme: StreamPollInteractorThemeData(
         pollTitleStyle: textTheme.headlineBold.copyWith(

packages/stream_chat_flutter/test/src/localization/default_translations_test.dart

@@ -76,6 +76,8 @@ void main() {
     expect(translations.deleteLabel, isNotNull);
     expect(translations.deleteMessageLabel, isNotNull);
     expect(translations.deleteMessageQuestion, isNotNull);
+    expect(translations.deletePollOptionLabel, isNotNull);
+    expect(translations.deletePollOptionQuestion, isNotNull);
     expect(translations.operationCouldNotBeCompletedText, isNotNull);
     expect(translations.replyLabel, isNotNull);
     // pinned