Merge branch 'soru/user-slider-on-pill-tap' into 'main'

feat: Open user bottom sheet on tapping user pills

See merge request famedly/fluffychat!529

lib/pages/user_bottom_sheet.dart

@@ -9,15 +9,18 @@ import 'package:future_loading_dialog/future_loading_dialog.dart';
 import 'package:vrouter/vrouter.dart';
 
 import 'views/user_bottom_sheet_view.dart';
+import '../widgets/matrix.dart';
 
 class UserBottomSheet extends StatefulWidget {
   final User user;
+  final Profile profile;
   final Function onMention;
   final BuildContext outerContext;
 
   const UserBottomSheet({
     Key key,
-    @required this.user,
+    this.user,
+    this.profile,
     @required this.outerContext,
     this.onMention,
   }) : super(key: key);
@@ -83,6 +86,7 @@ class UserBottomSheetController extends State<UserBottomSheet> {
         }
         break;
       case 'message':
+        if (widget.user != null) {
           final roomIdResult = await showFutureLoadingDialog(
             context: context,
             future: () => widget.user.startDirectChat(),
@@ -91,6 +95,19 @@ class UserBottomSheetController extends State<UserBottomSheet> {
           VRouter.of(widget.outerContext)
               .toSegments(['rooms', roomIdResult.result]);
           Navigator.of(context, rootNavigator: false).pop();
+        } else {
+          final result = await showFutureLoadingDialog<String>(
+            context: context,
+            future: () => Matrix.of(context).client.startDirectChat(
+                  widget.profile.userId,
+                ),
+          );
+          if (result.error == null) {
+            VRouter.of(context).toSegments(['rooms', result.result]);
+            Navigator.of(context, rootNavigator: false).pop();
+            return;
+          }
+        }
         break;
     }
   }

lib/pages/views/user_bottom_sheet_view.dart

@@ -5,6 +5,7 @@ import 'package:fluffychat/config/themes.dart';
 import 'package:fluffychat/utils/fluffy_share.dart';
 import 'package:flutter/material.dart';
 import '../../widgets/content_banner.dart';
+import '../../widgets/matrix.dart';
 import '../user_bottom_sheet.dart';
 import '../../utils/matrix_sdk_extensions.dart/presence_extension.dart';
 import 'package:flutter_gen/gen_l10n/l10n.dart';
@@ -17,8 +18,12 @@ class UserBottomSheetView extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     final user = controller.widget.user;
-    final client = user.room.client;
+    final client = user?.room?.client ?? Matrix.of(context).client;
-    final presence = client.presences[user.id];
+    final mxid = user?.id ?? controller.widget.profile?.userId ?? '';
+    final presence = client.presences[mxid];
+    final displayname =
+        user?.calcDisplayname() ?? controller.widget.profile?.displayName ?? '';
+    final avatarUrl = user?.avatarUrl ?? controller.widget.profile?.avatarUrl;
     return Center(
       child: Container(
         width: min(
@@ -37,12 +42,12 @@ class UserBottomSheetView extends StatelessWidget {
                   onPressed: Navigator.of(context, rootNavigator: false).pop,
                   tooltip: L10n.of(context).close,
                 ),
-                title: Text(user.calcDisplayname()),
+                title: Text(displayname),
                 actions: [
-                  if (user.id != user.room.client.userID)
+                  if (mxid != client.userID)
                     PopupMenuButton(
                       itemBuilder: (_) => [
-                        if (controller.widget.onMention != null)
+                        if (user != null && controller.widget.onMention != null)
                           PopupMenuItem(
                             value: 'mention',
                             child: _TextWithIcon(
@@ -50,8 +55,8 @@ class UserBottomSheetView extends StatelessWidget {
                               Icons.alternate_email_outlined,
                             ),
                           ),
-                        if (user.id != user.room.client.userID &&
+                        if (mxid != client.userID &&
-                            !user.room.isDirectChat)
+                            (user == null || !user.room.isDirectChat))
                           PopupMenuItem(
                             value: 'message',
                             child: _TextWithIcon(
@@ -59,7 +64,7 @@ class UserBottomSheetView extends StatelessWidget {
                               Icons.send_outlined,
                             ),
                           ),
-                        if (user.canChangePowerLevel)
+                        if (user != null && user.canChangePowerLevel)
                           PopupMenuItem(
                             value: 'permission',
                             child: _TextWithIcon(
@@ -67,7 +72,7 @@ class UserBottomSheetView extends StatelessWidget {
                               Icons.edit_attributes_outlined,
                             ),
                           ),
-                        if (user.canKick)
+                        if (user != null && user.canKick)
                           PopupMenuItem(
                             value: 'kick',
                             child: _TextWithIcon(
@@ -75,7 +80,9 @@ class UserBottomSheetView extends StatelessWidget {
                               Icons.exit_to_app_outlined,
                             ),
                           ),
-                        if (user.canBan && user.membership != Membership.ban)
+                        if (user != null &&
+                            user.canBan &&
+                            user.membership != Membership.ban)
                           PopupMenuItem(
                             value: 'ban',
                             child: _TextWithIcon(
@@ -83,7 +90,8 @@ class UserBottomSheetView extends StatelessWidget {
                               Icons.warning_sharp,
                             ),
                           )
-                        else if (user.canBan &&
+                        else if (user != null &&
+                            user.canBan &&
                             user.membership == Membership.ban)
                           PopupMenuItem(
                             value: 'unban',
@@ -101,17 +109,17 @@ class UserBottomSheetView extends StatelessWidget {
                 children: [
                   Expanded(
                     child: ContentBanner(
-                      user.avatarUrl,
+                      avatarUrl,
                       defaultIcon: Icons.person_outline,
-                      client: user.room.client,
+                      client: client,
                     ),
                   ),
                   ListTile(
                     title: Text(L10n.of(context).username),
-                    subtitle: Text(user.id),
+                    subtitle: Text(mxid),
                     trailing: Icon(Icons.share_outlined),
-                    onTap: () => FluffyShare.share(
+                    onTap: () =>
-                        user.id, controller.widget.outerContext),
+                        FluffyShare.share(mxid, controller.widget.outerContext),
                   ),
                   if (presence != null)
                     ListTile(

lib/utils/url_launcher.dart

@@ -11,6 +11,7 @@ import 'package:punycode/punycode.dart';
 import 'package:flutter_gen/gen_l10n/l10n.dart';
 
 import 'platform_infos.dart';
+import '../pages/user_bottom_sheet.dart';
 
 class UrlLauncher {
   final String url;
@@ -166,16 +167,15 @@ class UrlLauncher {
         });
       }
     } else if (identityParts.primaryIdentifier.sigil == '@') {
-      final result = await showFutureLoadingDialog<String>(
+      final profile = await matrix.client
+          .getProfileFromUserId(identityParts.primaryIdentifier);
+      await showModalBottomSheet(
         context: context,
-        future: () => matrix.client.startDirectChat(
+        builder: (c) => UserBottomSheet(
-          identityParts.primaryIdentifier,
+          profile: profile,
+          outerContext: context,
         ),
       );
-      if (result.error == null) {
-        VRouter.of(context).toSegments(['rooms', result.result]);
-        return;
-      }
     }
   }
 }