feat: update dependencies, ui and linting

Author: HayesGordon

packages/imessage/analysis_options.yaml

@@ -1,60 +1,29 @@
-include: package:pedantic/analysis_options.yaml
+# This file configures the analyzer, which statically analyzes Dart code to
+# check for errors, warnings, and lints.
+#
+# The issues identified by the analyzer are surfaced in the UI of Dart-enabled
+# IDEs (https://dart.dev/tools#ides-and-editors). The analyzer can also be
+# invoked from the command line by running `flutter analyze`.
 
-analyzer:
-  exclude:
-    - lib/**/*.g.dart
-    - example/*
-    - test/*
+# The following line activates a set of recommended lints for Flutter apps,
+# packages, and plugins designed to encourage good coding practices.
+include: package:flutter_lints/flutter.yaml
 
 linter:
+  # The lint rules applied to this project can be customized in the
+  # section below to disable rules from the `package:flutter_lints/flutter.yaml`
+  # included above or to enable additional rules. A list of all available lints
+  # and their documentation is published at
+  # https://dart-lang.github.io/linter/lints/index.html.
+  #
+  # Instead of disabling a lint rule for the entire project in the
+  # section below, it can also be suppressed for a single line of code
+  # or a specific dart file by using the `// ignore: name_of_lint` and
+  # `// ignore_for_file: name_of_lint` syntax on the line or in the file
+  # producing the lint.
   rules:
-    # these rules are documented on and in the same order as
-    # the Dart Lint rules page to make maintenance easier
-    # https://github.com/dart-lang/linter/blob/master/example/all.yaml
-    # - always_declare_return_types
-    # - always_specify_types
-    # - annotate_overrides
-    # - avoid_as
-    - avoid_empty_else
-    - avoid_init_to_null
-    - avoid_return_types_on_setters
-    - avoid_web_libraries_in_flutter
-    - await_only_futures
-    - camel_case_types
-    - cancel_subscriptions
-    - close_sinks
-    # - comment_references # we do not presume as to what people want to reference in their dartdocs
-    # - constant_identifier_names # https://github.com/dart-lang/linter/issues/204
-    - control_flow_in_finally
-    - empty_constructor_bodies
-    - empty_statements
-    - hash_and_equals
-    - implementation_imports
-    # - invariant_booleans
-    # - iterable_contains_unrelated_type
-    - library_names
-    # - library_prefixes
-    # - list_remove_unrelated_type
-    # - literal_only_boolean_expressions
-    - non_constant_identifier_names
-    # - one_member_abstracts
-    # - only_throw_errors
-    # - overridden_fields
-    - package_names
-    - package_prefixed_library_names
-    - prefer_is_not_empty
-    # - prefer_mixin # https://github.com/dart-lang/language/issues/32
-    - slash_for_doc_comments
-    # - sort_constructors_first
-    # - sort_unnamed_constructors_first
-    # - super_goes_last # no longer needed w/ Dart 2
-    - test_types_in_equals
-    - throw_in_finally
-    # - type_annotate_public_apis # subset of always_specify_types
-    - type_init_formals
-    # - unawaited_futures
-    - unnecessary_brace_in_string_interps
-    - unnecessary_getters_setters
-    - unnecessary_statements
-    - unrelated_type_equality_checks
-    - valid_regexps
+    # avoid_print: false  # Uncomment to disable the `avoid_print` rule
+    # prefer_single_quotes: true  # Uncomment to enable the `prefer_single_quotes` rule
+
+# Additional information about this file can be found at
+# https://dart.dev/guides/language/analysis-options

packages/imessage/lib/channel_image.dart

@@ -1,11 +1,15 @@
 import 'package:flutter/cupertino.dart';
-import 'package:stream_chat_flutter/stream_chat_flutter.dart' show Channel;
+import 'package:stream_chat_flutter_core/stream_chat_flutter_core.dart'
+    show Channel;
 
 import 'package:imessage/utils.dart';
 
 class ChannelImage extends StatelessWidget {
-  const ChannelImage({Key? key, required this.channel, required this.size})
-      : super(key: key);
+  const ChannelImage({
+    Key? key,
+    required this.channel,
+    required this.size,
+  }) : super(key: key);
 
   final Channel channel;
   final double size;

packages/imessage/lib/channel_list_view.dart

@@ -2,12 +2,13 @@ import 'package:flutter/cupertino.dart';
 import 'package:imessage/channel_preview.dart';
 import 'package:imessage/message_page.dart';
 import 'package:animations/animations.dart';
-import 'package:stream_chat_flutter/stream_chat_flutter.dart'
+import 'package:stream_chat_flutter_core/stream_chat_flutter_core.dart'
     show Channel, StreamChannel;
 
 class ChannelListView extends StatelessWidget {
   const ChannelListView({Key? key, required this.channels}) : super(key: key);
   final List<Channel> channels;
+
   @override
   Widget build(BuildContext context) {
     channels.removeWhere((channel) => channel.lastMessageAt == null);
@@ -27,7 +28,7 @@ class ChannelListView extends StatelessWidget {
                   PageRouteBuilder(
                     pageBuilder: (_, __, ___) => StreamChannel(
                       channel: channels[index],
-                      child: MessagePage(),
+                      child: const MessagePage(),
                     ),
                     transitionsBuilder: (
                       _,

packages/imessage/lib/channel_name_text.dart

@@ -1,23 +1,26 @@
 import 'package:flutter/cupertino.dart';
-import 'package:stream_chat_flutter/stream_chat_flutter.dart' show Channel;
+import 'package:stream_chat_flutter_core/stream_chat_flutter_core.dart'
+    show Channel;
 
 class ChannelNameText extends StatelessWidget {
   const ChannelNameText({
     Key? key,
     required this.channel,
     this.size = 17,
+    this.fontWeight,
   }) : super(key: key);
 
   final Channel channel;
   final double size;
+  final FontWeight? fontWeight;
 
   @override
   Widget build(BuildContext context) {
     return Text(
       channel.extraData['name'] as String? ?? 'No name',
       style: TextStyle(
         fontSize: size,
-        fontWeight: FontWeight.bold,
+        fontWeight: fontWeight,
         color: CupertinoColors.black,
       ),
     );

packages/imessage/lib/channel_page_appbar.dart

@@ -7,8 +7,11 @@ class ChannelPageAppBar extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    return CupertinoSliverNavigationBar(
-      largeTitle: Text('Messages'),
+    return const CupertinoSliverNavigationBar(
+      largeTitle: Text(
+        'Messages',
+        style: TextStyle(letterSpacing: -1.3),
+      ),
     );
   }
 }

packages/imessage/lib/channel_preview.dart

@@ -2,7 +2,8 @@ import 'package:flutter/cupertino.dart';
 import 'package:imessage/channel_image.dart';
 import 'package:imessage/channel_name_text.dart';
 import 'package:imessage/utils.dart';
-import 'package:stream_chat_flutter/stream_chat_flutter.dart' show Channel;
+import 'package:stream_chat_flutter_core/stream_chat_flutter_core.dart'
+    show Channel;
 
 import 'utils.dart';
 
@@ -23,7 +24,7 @@ class ChannelPreview extends StatelessWidget {
         : null;
 
     final prefix = lastMessage?.attachments != null
-        ? lastMessage?.attachments //TODO: ugly
+        ? lastMessage?.attachments
             .map((e) {
               if (e.type == 'image') {
                 return '📷 ';
@@ -35,13 +36,12 @@ class ChannelPreview extends StatelessWidget {
             .where((e) => e != null)
             .join(' ')
         : '';
+
     return GestureDetector(
       onTap: onTap,
       behavior: HitTestBehavior.opaque,
-      child: Container(
-        constraints: BoxConstraints.tightFor(
-          height: 90,
-        ),
+      child: SizedBox(
+        height: 70,
         child: Padding(
           padding: const EdgeInsets.symmetric(
             horizontal: 8.0,
@@ -50,11 +50,10 @@ class ChannelPreview extends StatelessWidget {
             crossAxisAlignment: CrossAxisAlignment.start,
             children: [
               Padding(
-                padding:
-                    const EdgeInsets.symmetric(vertical: 16.0, horizontal: 8.0),
+                padding: const EdgeInsets.only(left: 16.0, right: 8.0),
                 child: ChannelImage(
                   channel: channel,
-                  size: 50,
+                  size: 46,
                 ),
               ),
               Expanded(
@@ -67,47 +66,49 @@ class ChannelPreview extends StatelessWidget {
                       crossAxisAlignment: CrossAxisAlignment.start,
                       children: [
                         Padding(
-                          padding: const EdgeInsets.symmetric(horizontal: 8.0),
-                          child: ChannelNameText(
-                            channel: channel,
-                          ),
+                          padding: const EdgeInsets.symmetric(horizontal: 4.0),
+                          child: ChannelNameText(channel: channel),
                         ),
                         Padding(
-                          padding: const EdgeInsets.symmetric(horizontal: 8.0),
+                          padding: const EdgeInsets.symmetric(horizontal: 4.0),
                           child: Row(
                             children: [
                               Text(
                                 isSameWeek(channel.lastMessageAt!)
                                     ? formatDateSameWeek(channel.lastMessageAt!)
                                     : formatDate(channel.lastMessageAt!),
-                                style: TextStyle(
-                                  fontSize: 15,
+                                style: const TextStyle(
+                                  fontSize: 13,
                                   color: CupertinoColors.systemGrey,
                                 ),
                               ),
-                              Icon(
-                                CupertinoIcons.right_chevron,
-                                color: CupertinoColors.systemGrey3,
+                              const Padding(
+                                padding: EdgeInsets.only(left: 4.0),
+                                child: Icon(
+                                  CupertinoIcons.chevron_right,
+                                  size: 16,
+                                  color: CupertinoColors.systemGrey3,
+                                ),
                               ),
                             ],
                           ),
                         )
                       ],
                     ),
                     Padding(
-                      padding: const EdgeInsets.all(8.0),
+                      padding: const EdgeInsets.all(4.0),
                       child: Text(
                         '$prefix${lastMessage?.text ?? ''}',
-                        style: TextStyle(
+                        style: const TextStyle(
                           fontWeight: FontWeight.normal,
                           color: CupertinoColors.systemGrey,
-                          fontSize: 16,
+                          fontSize: 14,
                         ),
                         maxLines: 2,
                         overflow: TextOverflow.ellipsis,
                       ),
                     ),
-                    Divider(),
+                    const Divider(),
                   ],
                 ),
               )