lint changes

packages/stream_chat_flutter/lib/src/option_list_tile.dart

@@ -1,17 +1,11 @@
 import 'package:flutter/material.dart';
 import 'package:stream_chat_flutter/src/stream_chat_theme.dart';
 
+/// List tile for [ChannelBottomSheet]
 class OptionListTile extends StatelessWidget {
-  final String? title;
+  /// Constructor for creating [OptionListTile]
-  final Widget? leading;
-  final Widget? trailing;
-  final VoidCallback? onTap;
-  final Color? titleColor;
-  final Color? tileColor;
-  final Color? separatorColor;
-  final TextStyle? titleTextStyle;
-
   const OptionListTile({
+    Key? key,
     this.title,
     this.leading,
     this.trailing,
@@ -20,11 +14,34 @@ class OptionListTile extends StatelessWidget {
     this.tileColor,
     this.separatorColor,
     this.titleTextStyle,
-  });
+  }) : super(key: key);
+
+  /// Title for tile
+  final String? title;
+
+  /// Leading widget (start)
+  final Widget? leading;
+
+  /// Trailing widget (end)
+  final Widget? trailing;
+
+  /// Callback when tile is tapped
+  final VoidCallback? onTap;
+
+  /// Title color
+  final Color? titleColor;
+
+  /// Background tile color
+  final Color? tileColor;
+
+  /// Separator color
+  final Color? separatorColor;
+
+  /// [TextStyle] to apply to [title]
+  final TextStyle? titleTextStyle;
 
   @override
-  Widget build(BuildContext context) {
+  Widget build(BuildContext context) => Column(
-    return Column(
         children: [
           Container(
             color: separatorColor ??
@@ -33,7 +50,7 @@ class OptionListTile extends StatelessWidget {
           ),
           Material(
             color: tileColor ?? StreamChatTheme.of(context).colorTheme.white,
-          child: Container(
+            child: SizedBox(
               height: 63,
               child: InkWell(
                 onTap: onTap,
@@ -41,7 +58,7 @@ class OptionListTile extends StatelessWidget {
                   children: [
                     if (leading != null) Center(child: leading),
                     if (leading == null)
-                    SizedBox(
+                      const SizedBox(
                         width: 16,
                       ),
                     Expanded(
@@ -77,4 +94,3 @@ class OptionListTile extends StatelessWidget {
         ],
       );
 }
-}

packages/stream_chat_flutter/lib/src/quoted_message_widget.dart

@@ -4,29 +4,25 @@ import 'package:cached_network_image/cached_network_image.dart';
 import 'package:flutter/material.dart';
 import 'package:stream_chat_flutter_core/stream_chat_flutter_core.dart';
 import 'package:video_player/video_player.dart';
+import 'package:stream_chat_flutter/stream_chat_flutter.dart';
+import 'package:stream_chat_flutter/src/extension.dart';
 
-import 'attachment/attachment.dart';
+/// Widget builder for quoted message attachment thumnail
-import 'extension.dart';
-import 'message_text.dart';
-import 'stream_chat_theme.dart';
-import 'user_avatar.dart';
-import 'utils.dart';
-
 typedef QuotedMessageAttachmentThumbnailBuilder = Widget Function(
   BuildContext,
   Attachment,
 );
 
 class _VideoAttachmentThumbnail extends StatefulWidget {
-  final Size size;
-  final Attachment attachment;
-
   const _VideoAttachmentThumbnail({
     Key? key,
     required this.attachment,
     this.size = const Size(32, 32),
   }) : super(key: key);
 
+  final Size size;
+  final Attachment attachment;
+
   @override
   _VideoAttachmentThumbnailState createState() =>
       _VideoAttachmentThumbnailState();
@@ -51,18 +47,30 @@ class _VideoAttachmentThumbnailState extends State<_VideoAttachmentThumbnail> {
   }
 
   @override
-  Widget build(BuildContext context) {
+  Widget build(BuildContext context) => SizedBox(
-    return Container(
         height: widget.size.height,
         width: widget.size.width,
         child: _controller.value.isInitialized
             ? VideoPlayer(_controller)
-            : CircularProgressIndicator());
+            : const CircularProgressIndicator(),
-  }
+      );
 }
 
 ///
 class QuotedMessageWidget extends StatelessWidget {
+  ///
+  const QuotedMessageWidget({
+    Key? key,
+    required this.message,
+    required this.messageTheme,
+    this.reverse = false,
+    this.showBorder = false,
+    this.textLimit = 170,
+    this.attachmentThumbnailBuilders,
+    this.padding = const EdgeInsets.all(8),
+    this.onTap,
+  }) : super(key: key);
+
   /// The message
   final Message message;
 
@@ -82,23 +90,12 @@ class QuotedMessageWidget extends StatelessWidget {
   final Map<String, QuotedMessageAttachmentThumbnailBuilder>?
       attachmentThumbnailBuilders;
 
+  /// Padding around the widget
   final EdgeInsetsGeometry padding;
 
+  /// Callback for tap on widget
   final GestureTapCallback? onTap;
 
-  ///
-  const QuotedMessageWidget({
-    Key? key,
-    required this.message,
-    required this.messageTheme,
-    this.reverse = false,
-    this.showBorder = false,
-    this.textLimit = 170,
-    this.attachmentThumbnailBuilders,
-    this.padding = const EdgeInsets.all(8),
-    this.onTap,
-  }) : super(key: key);
-
   bool get _hasAttachments => message.attachments.isNotEmpty == true;
 
   bool get _containsScrapeUrl =>
@@ -107,8 +104,7 @@ class QuotedMessageWidget extends StatelessWidget {
   bool get _containsText => message.text?.isNotEmpty == true;
 
   @override
-  Widget build(BuildContext context) {
+  Widget build(BuildContext context) => Padding(
-    return Padding(
         padding: padding,
         child: InkWell(
           onTap: onTap,
@@ -117,13 +113,12 @@ class QuotedMessageWidget extends StatelessWidget {
             mainAxisSize: MainAxisSize.min,
             children: [
               Flexible(child: _buildMessage(context)),
-            SizedBox(width: 8),
+              const SizedBox(width: 8),
               if (message.user != null) _buildUserAvatar(),
             ],
           ),
         ),
       );
-  }
 
   Widget _buildMessage(BuildContext context) {
     final isOnlyEmoji = message.text!.isOnlyEmoji;
@@ -165,7 +160,7 @@ class QuotedMessageWidget extends StatelessWidget {
                 color: StreamChatTheme.of(context).colorTheme.greyGainsboro,
               )
             : null,
-        borderRadius: BorderRadius.only(
+        borderRadius: const BorderRadius.only(
           topRight: Radius.circular(12),
           topLeft: Radius.circular(12),
           bottomLeft: Radius.circular(12),
@@ -183,7 +178,7 @@ class QuotedMessageWidget extends StatelessWidget {
   }
 
   Widget _buildUrlAttachment(Attachment attachment) {
-    final size = Size(32, 32);
+    const size = Size(32, 32);
     if (attachment.thumbUrl != null) {
       return Container(
         height: size.height,
@@ -198,7 +193,7 @@ class QuotedMessageWidget extends StatelessWidget {
         ),
       );
     }
-    return AttachmentError(size: size);
+    return const AttachmentError(size: size);
   }
 
   Widget _parseAttachments(BuildContext context) {
@@ -217,7 +212,7 @@ class QuotedMessageWidget extends StatelessWidget {
       }
       attachmentBuilder = _defaultAttachmentBuilder[attachment.type];
       if (attachmentBuilder == null) {
-        child = Offstage();
+        child = const Offstage();
       } else {
         child = attachmentBuilder(context, attachment);
       }
@@ -235,77 +230,63 @@ class QuotedMessageWidget extends StatelessWidget {
     );
   }
 
-  ShapeBorder _getDefaultShape(BuildContext context) {
+  ShapeBorder _getDefaultShape(BuildContext context) => RoundedRectangleBorder(
-    return RoundedRectangleBorder(
+        side: const BorderSide(width: 0, color: Colors.transparent),
-      side: BorderSide(width: 0, color: Colors.transparent),
         borderRadius: BorderRadius.circular(8),
       );
-  }
 
-  Widget _buildUserAvatar() {
+  Widget _buildUserAvatar() => Transform(
-    return Transform(
         transform: Matrix4.rotationY(reverse ? pi : 0),
         alignment: Alignment.center,
         child: UserAvatar(
           user: message.user!,
-        constraints: BoxConstraints.tightFor(
+          constraints: const BoxConstraints.tightFor(
             height: 24,
             width: 24,
           ),
           showOnlineStatus: false,
         ),
       );
-  }
 
   Map<String, QuotedMessageAttachmentThumbnailBuilder>
-      get _defaultAttachmentBuilder {
+      get _defaultAttachmentBuilder => {
-    return {
+            'image': (_, attachment) => ImageAttachment(
-      'image': (_, attachment) {
-        return ImageAttachment(
                   attachment: attachment,
                   message: message,
                   messageTheme: messageTheme,
-          size: Size(32, 32),
+                  size: const Size(32, 32),
-        );
+                ),
-      },
+            'video': (_, attachment) => _VideoAttachmentThumbnail(
-      'video': (_, attachment) {
-        return _VideoAttachmentThumbnail(
                   key: ValueKey(attachment.assetUrl),
                   attachment: attachment,
-        );
+                ),
-      },
             'giphy': (_, attachment) {
-        final size = Size(32, 32);
+              const size = Size(32, 32);
               return CachedNetworkImage(
                 height: size.height,
                 width: size.width,
-          placeholder: (_, __) {
+                placeholder: (_, __) => SizedBox(
-            return Container(
                   width: size.width,
                   height: size.height,
-              child: Center(
+                  child: const Center(
                     child: CircularProgressIndicator(),
                   ),
-            );
+                ),
-          },
                 imageUrl: attachment.thumbUrl ??
                     attachment.imageUrl ??
                     attachment.assetUrl!,
-          errorWidget: (context, url, error) {
+                errorWidget: (context, url, error) =>
-            return AttachmentError(size: size);
+                    const AttachmentError(size: size),
-          },
                 fit: BoxFit.cover,
               );
             },
-      'file': (_, attachment) {
+            'file': (_, attachment) => SizedBox(
-        return Container(
                   height: 32,
                   width: 32,
-          child: getFileTypeImage(attachment.extraData['mime_type'] as String?),
+                  child: getFileTypeImage(
-        );
+                      attachment.extraData['mime_type'] as String?),
-      },
+                ),
           };
-  }
 
   Color? _getBackgroundColor(BuildContext context) {
     if (_containsScrapeUrl) {