diff --git a/packages/stream_chat_flutter/CHANGELOG.md b/packages/stream_chat_flutter/CHANGELOG.md index 840d02a4..6c3980d2 100644 --- a/packages/stream_chat_flutter/CHANGELOG.md +++ b/packages/stream_chat_flutter/CHANGELOG.md @@ -9,6 +9,7 @@ 🐞 Fixed - [[#1379]](https://github.com/GetStream/stream-chat-flutter/issues/1379) Fixed "Issues with photo attachments on web", where the cached image attachment would not render while uploading. - Fix render overflow issue with `MessageSearchListTileTitle`. It now uses `Text.rich` instead of `Row`. Better default behaviour and allows `TextOverflow`. +- [[#1347]](https://github.com/GetStream/stream-chat-flutter/issues/1347) `onReply` not working in `AttachmentActionsModal` which is used by `StreamImageAttachment` and `StreamImageGroup`. ## 5.1.0 diff --git a/packages/stream_chat_flutter/example/lib/main.dart b/packages/stream_chat_flutter/example/lib/main.dart index 67420d06..c8978a45 100644 --- a/packages/stream_chat_flutter/example/lib/main.dart +++ b/packages/stream_chat_flutter/example/lib/main.dart @@ -300,6 +300,7 @@ class _ChannelPageState extends State { @override void dispose() { focusNode.dispose(); + messageInputController.dispose(); super.dispose(); } } diff --git a/packages/stream_chat_flutter/lib/src/attachment_actions_modal/attachment_actions_modal.dart b/packages/stream_chat_flutter/lib/src/attachment_actions_modal/attachment_actions_modal.dart index 510d2fe4..47a62a32 100644 --- a/packages/stream_chat_flutter/lib/src/attachment_actions_modal/attachment_actions_modal.dart +++ b/packages/stream_chat_flutter/lib/src/attachment_actions_modal/attachment_actions_modal.dart @@ -33,10 +33,10 @@ class AttachmentActionsModal extends StatelessWidget { /// Callback to download [attachment]. final AttachmentDownloader? attachmentDownloader; - /// Show reply option + /// Show "reply" option final bool showReply; - /// Show show in chat option + /// Show "show in chat" option final bool showShowInChat; /// Show save option @@ -55,6 +55,7 @@ class AttachmentActionsModal extends StatelessWidget { Attachment? attachment, Message? message, VoidCallback? onShowMessage, + VoidCallback? onReply, AttachmentDownloader? attachmentDownloader, bool? showReply, bool? showShowInChat, @@ -67,6 +68,7 @@ class AttachmentActionsModal extends StatelessWidget { attachment: attachment ?? this.attachment, message: message ?? this.message, onShowMessage: onShowMessage ?? this.onShowMessage, + onReply: onReply ?? this.onReply, attachmentDownloader: attachmentDownloader ?? this.attachmentDownloader, showReply: showReply ?? this.showReply, showShowInChat: showShowInChat ?? this.showShowInChat, @@ -112,7 +114,7 @@ class AttachmentActionsModal extends StatelessWidget { size: 24, color: theme.colorTheme.textLowEmphasis, ), - () => Navigator.of(context).pop(ReturnActionType.reply), + onReply, ), if (showShowInChat) _buildButton( diff --git a/packages/stream_chat_flutter/lib/src/fullscreen_media/fsm_enums.dart b/packages/stream_chat_flutter/lib/src/fullscreen_media/fsm_enums.dart index 5d2a1412..854682e1 100644 --- a/packages/stream_chat_flutter/lib/src/fullscreen_media/fsm_enums.dart +++ b/packages/stream_chat_flutter/lib/src/fullscreen_media/fsm_enums.dart @@ -1,4 +1,8 @@ +// TODO: remove in v6 as this is no longer used. Currently exported. + /// Return action for coming back from pages +@Deprecated(''' + ReturnActionType has been deprecated and is no longer used.''') enum ReturnActionType { /// No return action none,