chore: Follow up chat context menu

lib/pages/chat_list/chat_list.dart

@@ -620,6 +620,10 @@ class ChatListController extends State<ChatList>
     BuildContext posContext, [
     Room? space,
   ]) async {
+    if (room.membership == Membership.invite) {
+      return onChatTap(room);
+    }
+
     final overlay =
         Overlay.of(posContext).context.findRenderObject() as RenderBox;
 
@@ -644,7 +648,7 @@ class ChatListController extends State<ChatList>
       position: position,
       items: [
         PopupMenuItem(
-          enabled: false,
+          value: ChatContextAction.open,
           child: Row(
             mainAxisSize: MainAxisSize.min,
             children: [
@@ -754,12 +758,36 @@ class ChatListController extends State<ChatList>
     if (action == null) return;
     if (!mounted) return;
 
-    if (action == ChatContextAction.goToSpace) {
+    switch (action) {
+      case ChatContextAction.open:
+        onChatTap(room);
+        return;
+      case ChatContextAction.goToSpace:
         setActiveSpace(space!.id);
         return;
-    }
+      case ChatContextAction.favorite:
-
+        await showFutureLoadingDialog(
-    if (action == ChatContextAction.leave) {
+          context: context,
+          future: () => room.setFavourite(!room.isFavourite),
+        );
+        return;
+      case ChatContextAction.markUnread:
+        await showFutureLoadingDialog(
+          context: context,
+          future: () => room.markUnread(!room.markedUnread),
+        );
+        return;
+      case ChatContextAction.mute:
+        await showFutureLoadingDialog(
+          context: context,
+          future: () => room.setPushRuleState(
+            room.pushRuleState == PushRuleState.notify
+                ? PushRuleState.mentionsOnly
+                : PushRuleState.notify,
+          ),
+        );
+        return;
+      case ChatContextAction.leave:
         final confirmed = await showOkCancelAlertDialog(
           useRootNavigator: false,
           context: context,
@@ -770,30 +798,11 @@ class ChatListController extends State<ChatList>
           isDestructiveAction: true,
         );
         if (confirmed == OkCancelResult.cancel) return;
+        if (!mounted) {
+          await showFutureLoadingDialog(context: context, future: room.leave);
         }
-    if (!mounted) return;
-
-    await showFutureLoadingDialog(
-      context: context,
-      future: () async {
-        switch (action) {
-          case ChatContextAction.goToSpace:
         return;
-          case ChatContextAction.favorite:
-            return room.setFavourite(!room.isFavourite);
-          case ChatContextAction.markUnread:
-            return room.markUnread(!room.markedUnread);
-          case ChatContextAction.mute:
-            return room.setPushRuleState(
-              room.pushRuleState == PushRuleState.notify
-                  ? PushRuleState.mentionsOnly
-                  : PushRuleState.notify,
-            );
-          case ChatContextAction.leave:
-            return room.leave();
     }
-      },
-    );
   }
 
   void dismissStatusList() async {
@@ -993,6 +1002,7 @@ enum InviteActions {
 enum AddRoomType { chat, subspace }
 
 enum ChatContextAction {
+  open,
   goToSpace,
   favorite,
   markUnread,