Merge pull request #1381 from GetStream/fix/image-attachment-web

packages/stream_chat_flutter/CHANGELOG.md

@@ -1,7 +1,9 @@
 ## Upcomming
 
 🐞 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`.
+
 ## 5.1.0
 
 🐞 Fixed

packages/stream_chat_flutter/lib/src/attachment/image_attachment.dart

@@ -56,22 +56,33 @@ class StreamImageAttachment extends StreamAttachmentWidget {
   Widget build(BuildContext context) {
     return source.when(
       local: () {
-        if (attachment.localUri == null || attachment.file?.bytes == null) {
+        if (attachment.file?.bytes != null) {
-          return AttachmentError(constraints: constraints);
+          return _buildImageAttachment(
-        }
+            context,
-        return _buildImageAttachment(
+            Image.memory(
-          context,
+              attachment.file!.bytes!,
-          Image.memory(
+              height: constraints?.maxHeight,
-            attachment.file!.bytes!,
+              width: constraints?.maxWidth,
-            height: constraints?.maxHeight,
+              fit: BoxFit.cover,
-            width: constraints?.maxWidth,
+              errorBuilder: _imageErrorBuilder,
-            fit: BoxFit.cover,
-            errorBuilder: (context, _, __) => Image.asset(
-              'images/placeholder.png',
-              package: 'stream_chat_flutter',
             ),
-          ),
+          );
-        );
+        } else if (attachment.localUri != null) {
+          return _buildImageAttachment(
+            context,
+            Image.asset(
+              attachment.localUri!.path,
+              height: constraints?.maxHeight,
+              width: constraints?.maxWidth,
+              fit: BoxFit.cover,
+              errorBuilder: _imageErrorBuilder,
+            ),
+          );
+        } else {
+          return AttachmentError(
+            constraints: constraints,
+          );
+        }
       },
       network: () {
         var imageUrl =
@@ -116,6 +127,12 @@ class StreamImageAttachment extends StreamAttachmentWidget {
     );
   }
 
+  Widget _imageErrorBuilder(BuildContext _, Object __, StackTrace? ___) =>
+      Image.asset(
+        'images/placeholder.png',
+        package: 'stream_chat_flutter',
+      );
+
   Widget _buildImageAttachment(BuildContext context, Widget imageWidget) {
     return Container(
       constraints: constraints,