From 518bc02d2cfd17043bc47b168f17baa0189e4c5d Mon Sep 17 00:00:00 2001
From: Sahil Kumar <xdsahil@gmail.com>
Date: Fri, 19 May 2023 17:34:49 +0530
Subject: [PATCH] fix(ui): handle image load error for url attachments.

Signed-off-by: xsahil03x <xdsahil@gmail.com>
---
 .../lib/src/attachment/url_attachment.dart    | 55 +++++++++++++++++--
 1 file changed, 51 insertions(+), 4 deletions(-)

diff --git a/packages/stream_chat_flutter/lib/src/attachment/url_attachment.dart b/packages/stream_chat_flutter/lib/src/attachment/url_attachment.dart
index a8ed00c6..968803d0 100644
--- a/packages/stream_chat_flutter/lib/src/attachment/url_attachment.dart
+++ b/packages/stream_chat_flutter/lib/src/attachment/url_attachment.dart
@@ -1,5 +1,6 @@
 import 'package:cached_network_image/cached_network_image.dart';
 import 'package:flutter/material.dart';
+import 'package:shimmer/shimmer.dart';
 import 'package:stream_chat_flutter/stream_chat_flutter.dart';
 
 /// {@template streamUrlAttachment}
@@ -34,6 +35,33 @@ class StreamUrlAttachment extends StatelessWidget {
   /// The function called when tapping on a link
   final void Function(String)? onLinkTap;
 
+  //placeholder: (context, __) {
+  //               final image = Image.asset(
+  //                 'images/placeholder.png',
+  //                 fit: BoxFit.cover,
+  //                 package: 'stream_chat_flutter',
+  //               );
+  //               final colorTheme = StreamChatTheme.of(context).colorTheme;
+  //               return Shimmer.fromColors(
+  //                 baseColor: colorTheme.disabled,
+  //                 highlightColor: colorTheme.inputBg,
+  //                 child: image,
+  //               );
+  //             },
+  //             errorWidget: (context, url, error) =>
+  //                 AttachmentError(constraints: constraints),
+  //           ),
+  //         );
+  //       },
+  //     );
+  //   }
+  //
+  Widget _imageErrorBuilder(BuildContext _, Object __, StackTrace? ___) =>
+      Image.asset(
+        'images/placeholder.png',
+        package: 'stream_chat_flutter',
+      );
+
   @override
   Widget build(BuildContext context) {
     return ConstrainedBox(
@@ -64,10 +92,29 @@ class StreamUrlAttachment extends StatelessWidget {
                   ),
                   child: Stack(
                     children: [
-                      CachedNetworkImage(
-                        width: double.infinity,
-                        imageUrl: urlAttachment.imageUrl!,
-                        fit: BoxFit.cover,
+                      AspectRatio(
+                        // Default aspect ratio for Open Graph images.
+                        // https://www.kapwing.com/resources/what-is-an-og-image-make-and-format-og-images-for-your-blog-or-webpage
+                        aspectRatio: 1.91 / 1,
+                        child: CachedNetworkImage(
+                          imageUrl: urlAttachment.imageUrl!,
+                          fit: BoxFit.cover,
+                          placeholder: (context, __) {
+                            final image = Image.asset(
+                              'images/placeholder.png',
+                              fit: BoxFit.cover,
+                              package: 'stream_chat_flutter',
+                            );
+                            final colorTheme =
+                                StreamChatTheme.of(context).colorTheme;
+                            return Shimmer.fromColors(
+                              baseColor: colorTheme.disabled,
+                              highlightColor: colorTheme.inputBg,
+                              child: image,
+                            );
+                          },
+                          errorWidget: (_, __, ___) => const AttachmentError(),
+                        ),
                       ),
                       Positioned(
                         left: 0,