feat: Add Label as to context menu for each email item in list

2026-02-04 16:45:45 +07:00
parent 0a88a8f109
commit ff4515c26a
5 changed files with 105 additions and 50 deletions
@@ -45,7 +45,6 @@ import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_i
 import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_read_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_star_email_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/print_email_interactor.dart';
-import 'package:tmail_ui_user/features/email/presentation/extensions/presentation_email_extension.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/context_item_email_action.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/email_loaded.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/email_unsubscribe.dart';
@@ -53,8 +52,8 @@ import 'package:tmail_ui_user/features/email/presentation/model/popup_menu_item_
 import 'package:tmail_ui_user/features/email/presentation/utils/email_utils.dart';
 import 'package:tmail_ui_user/features/email/presentation/widgets/email_address_bottom_sheet_builder.dart';
 import 'package:tmail_ui_user/features/email/presentation/widgets/email_address_dialog_builder.dart';
+import 'package:tmail_ui_user/features/labels/presentation/mixin/label_sub_menu_mixin.dart';
 import 'package:tmail_ui_user/features/labels/presentation/widgets/label_item_context_menu.dart';
-import 'package:tmail_ui_user/features/labels/presentation/widgets/label_list_context_menu.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/model/mailbox_actions.dart';
 import 'package:tmail_ui_user/features/manage_account/domain/model/create_new_email_rule_filter_request.dart';
 import 'package:tmail_ui_user/features/manage_account/domain/usecases/create_new_email_rule_filter_interactor.dart';
@@ -82,7 +81,7 @@ typedef OpenPopupMenuActionGroup = Future<void> Function(
  PopupMenuActionGroupWidget popupMenuWidget,
 );

-class EmailActionReactor {
+class EmailActionReactor with LabelSubMenuMixin {
  EmailActionReactor(
    this._markAsEmailReadInteractor,
    this._markAsStarEmailInteractor,
@@ -575,7 +574,7 @@ class EmailActionReactor {
          imagePaths,
          key: '${actionType.name}_action',
          category: actionType.category,
-          submenu: _getEmailActionSubmenu(
+          submenu: buildLabelSubmenuForEmail(
            actionType: actionType,
            imagePaths: imagePaths,
            presentationEmail: presentationEmail,
@@ -593,7 +592,7 @@ class EmailActionReactor {
        actions: popupMenuItemEmailActions,
        submenuController: submenuController,
        onActionSelected: (action) {
-          if (_shouldHandleAction(action.action)) {
+          if (shouldHandleAction(action.action)) {
            handleEmailAction(presentationEmail, action.action);
          }
        },
@@ -607,36 +606,6 @@ class EmailActionReactor {
    }
  }

-  bool _shouldHandleAction(EmailActionType action) {
-    if (action != EmailActionType.labelAs) {
-      return true;
-    }
-
-    return PlatformInfo.isWebTouchDevice || PlatformInfo.isMobile;
-  }
-
-  Widget? _getEmailActionSubmenu({
-    required EmailActionType actionType,
-    required ImagePaths imagePaths,
-    required PresentationEmail presentationEmail,
-    required List<Label>? labels,
-    OnSelectLabelAction? onSelectLabelAction,
-  }) {
-    if (actionType == EmailActionType.labelAs && labels?.isNotEmpty == true) {
-      final listLabels = labels ?? [];
-      final emailLabels = presentationEmail.getLabelList(listLabels);
-
-      return LabelListContextMenu(
-        labelList: listLabels,
-        emailLabels: emailLabels,
-        imagePaths: imagePaths,
-        onSelectLabelAction: (label, isSelected) =>
-            onSelectLabelAction?.call(label, isSelected),
-      );
-    }
-    return null;
-  }
-
  bool _canDeletePermanently(
    PresentationEmail email,
    PresentationMailbox? mailboxContain,
@@ -0,0 +1,38 @@
+import 'package:core/presentation/resources/image_paths.dart';
+import 'package:core/utils/platform_info.dart';
+import 'package:flutter/material.dart';
+import 'package:labels/model/label.dart';
+import 'package:model/email/email_action_type.dart';
+import 'package:model/email/presentation_email.dart';
+import 'package:tmail_ui_user/features/email/presentation/extensions/presentation_email_extension.dart';
+import 'package:tmail_ui_user/features/labels/presentation/widgets/label_item_context_menu.dart';
+import 'package:tmail_ui_user/features/labels/presentation/widgets/label_list_context_menu.dart';
+
+mixin LabelSubMenuMixin {
+  Widget? buildLabelSubmenuForEmail({
+    required EmailActionType actionType,
+    required ImagePaths imagePaths,
+    required PresentationEmail presentationEmail,
+    required List<Label>? labels,
+    required OnSelectLabelAction onSelectLabelAction,
+  }) {
+    if (actionType == EmailActionType.labelAs && labels?.isNotEmpty == true) {
+      final listLabels = labels ?? [];
+      final emailLabels = presentationEmail.getLabelList(listLabels);
+      return LabelListContextMenu(
+        labelList: listLabels,
+        emailLabels: emailLabels,
+        imagePaths: imagePaths,
+        onSelectLabelAction: onSelectLabelAction,
+      );
+    }
+    return null;
+  }
+
+  bool shouldHandleAction(EmailActionType action) {
+    if (action != EmailActionType.labelAs) {
+      return true;
+    }
+    return PlatformInfo.isWebTouchDevice || PlatformInfo.isMobile;
+  }
+}
@@ -3,8 +3,10 @@ import 'package:model/email/email_action_type.dart';
 import 'package:model/email/presentation_email.dart';
 import 'package:model/extensions/presentation_mailbox_extension.dart';
 import 'package:tmail_ui_user/features/base/widget/popup_menu/popup_menu_action_group_widget.dart';
+import 'package:tmail_ui_user/features/base/widget/popup_menu/popup_submenu_controller.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/context_item_email_action.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/popup_menu_item_email_action.dart';
+import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/extensions/labels/handle_logic_label_extension.dart';
 import 'package:tmail_ui_user/features/search/email/presentation/search_email_controller.dart';
 import 'package:tmail_ui_user/main/localizations/app_localizations.dart';
 import 'package:tmail_ui_user/main/routes/route_navigation.dart';
@@ -21,10 +23,14 @@ extension HandleEmailMoreActionExtension on SearchEmailController {
    final isRead = presentationEmail.hasRead;
    final isTrash = mailboxContain?.isTrash ?? false;
    final canPermanentlyDelete = isDrafts || isSpam || isTrash;
+    final isLabelAvailable = mailboxDashBoardController.isLabelAvailable;
+    final listLabels = mailboxDashBoardController.labelController.labels;
+    final shouldShowLabelAs = isLabelAvailable && listLabels.isNotEmpty;

    final listEmailActions = [
      isRead ? EmailActionType.markAsUnread : EmailActionType.markAsRead,
      EmailActionType.moveToMailbox,
+      if (shouldShowLabelAs) EmailActionType.labelAs,
      canPermanentlyDelete ? EmailActionType.deletePermanently : EmailActionType.moveToTrash,
      isSpam ? EmailActionType.unSpam : EmailActionType.moveToSpam,
      if (!isDrafts) EmailActionType.editAsNewEmail,
@@ -57,31 +63,46 @@ extension HandleEmailMoreActionExtension on SearchEmailController {
        useGroupedActions: true,
      );
    } else {
+      final submenuController = PopupSubmenuController();
+
      final popupMenuItemEmailActions = listEmailActions.map((actionType) {
        return PopupMenuItemEmailAction(
          actionType,
          AppLocalizations.of(context),
          imagePaths,
          category: actionType.category,
+          submenu: buildLabelSubmenuForEmail(
+            actionType: actionType,
+            imagePaths: imagePaths,
+            presentationEmail: presentationEmail,
+            labels: listLabels,
+            onSelectLabelAction: (label, isSelected) {
+              submenuController.hide();
+              popBack();
+            },
+          ),
        );
      }).toList();

      final popupMenuWidget = PopupMenuActionGroupWidget(
        actions: popupMenuItemEmailActions,
+        submenuController: submenuController,
        onActionSelected: (action) {
-          pressEmailAction(
-            context,
-            action.action,
-            presentationEmail,
-            mailboxContain: mailboxContain,
-          );
+          if (shouldHandleAction(action.action)) {
+            pressEmailAction(
+              context,
+              action.action,
+              presentationEmail,
+              mailboxContain: mailboxContain,
+            );
+          }
        },
      );

      return popupMenuWidget.show(
        context,
        position,
-      );
+      ).whenComplete(submenuController.hide);
    }
  }
 }
@@ -40,6 +40,7 @@ import 'package:tmail_ui_user/features/email/presentation/action/email_ui_action
 import 'package:tmail_ui_user/features/email/presentation/utils/email_utils.dart';
 import 'package:tmail_ui_user/features/home/data/exceptions/session_exceptions.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/extensions/presentation_mailbox_extension.dart';
+import 'package:tmail_ui_user/features/labels/presentation/mixin/label_sub_menu_mixin.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/model/mailbox_actions.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/domain/model/recent_search.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/domain/state/get_all_recent_search_latest_state.dart';
@@ -88,7 +89,8 @@ import 'package:tmail_ui_user/main/routes/route_utils.dart';
 class SearchEmailController extends BaseController
    with EmailActionController,
        DateRangePickerMixin,
-        SearchLabelFilterModalMixin {
+        SearchLabelFilterModalMixin,
+        LabelSubMenuMixin {

  final networkConnectionController = Get.find<NetworkConnectionController>();

@@ -13,11 +13,13 @@ import 'package:tmail_ui_user/features/base/widget/clean_messages_banner.dart';
 import 'package:tmail_ui_user/features/base/widget/compose_floating_button.dart';
 import 'package:tmail_ui_user/features/base/widget/keyboard/keyboard_handler_wrapper.dart';
 import 'package:tmail_ui_user/features/base/widget/popup_menu/popup_menu_action_group_widget.dart';
+import 'package:tmail_ui_user/features/base/widget/popup_menu/popup_submenu_controller.dart';
 import 'package:tmail_ui_user/features/base/widget/report_message_banner.dart';
 import 'package:tmail_ui_user/features/email/presentation/extensions/presentation_email_extension.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/composer_arguments.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/context_item_email_action.dart';
 import 'package:tmail_ui_user/features/email/presentation/model/popup_menu_item_email_action.dart';
+import 'package:tmail_ui_user/features/labels/presentation/mixin/label_sub_menu_mixin.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/state/clear_mailbox_state.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/state/mark_as_mailbox_read_state.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/state/move_folder_content_state.dart';
@@ -60,7 +62,9 @@ import 'package:tmail_ui_user/main/localizations/app_localizations.dart';
 import 'package:tmail_ui_user/main/routes/route_navigation.dart';

 class ThreadView extends GetWidget<ThreadController>
-  with AppLoaderMixin, PopupMenuWidgetMixin {
+  with AppLoaderMixin,
+      PopupMenuWidgetMixin,
+      LabelSubMenuMixin {

  ThreadView({Key? key}) : super(key: key);

@@ -689,10 +693,16 @@ class ThreadView extends GetWidget<ThreadController>
    final isRead = presentationEmail.hasRead;
    final isTrash = mailboxContain?.isTrash ?? false;
    final canPermanentlyDelete = isDrafts || isSpam || isTrash;
+    final isLabelAvailable =
+        controller.mailboxDashBoardController.isLabelAvailable;
+    final listLabels =
+        controller.mailboxDashBoardController.labelController.labels;
+    final shouldShowLabelAs = isLabelAvailable && listLabels.isNotEmpty;

    final listEmailActions = [
      isRead ? EmailActionType.markAsUnread : EmailActionType.markAsRead,
      EmailActionType.moveToMailbox,
+      if (shouldShowLabelAs) EmailActionType.labelAs,
      canPermanentlyDelete ? EmailActionType.deletePermanently : EmailActionType.moveToTrash,
      EmailActionType.openInNewTab,
      if (!isDrafts && !isChildOfTeamMailboxes)
@@ -725,23 +735,38 @@ class ThreadView extends GetWidget<ThreadController>
        useGroupedActions: true,
      );
    } else {
+      final submenuController = PopupSubmenuController();
+
      final popupMenuItemEmailActions = listEmailActions.map((actionType) {
        return PopupMenuItemEmailAction(
          actionType,
          AppLocalizations.of(context),
          controller.imagePaths,
          category: actionType.category,
+          submenu: buildLabelSubmenuForEmail(
+            actionType: actionType,
+            imagePaths: controller.imagePaths,
+            presentationEmail: presentationEmail,
+            labels: listLabels,
+            onSelectLabelAction: (label, isSelected) {
+              submenuController.hide();
+              popBack();
+            },
+          ),
        );
      }).toList();

      final popupMenuWidget = PopupMenuActionGroupWidget(
        actions: popupMenuItemEmailActions,
+        submenuController: submenuController,
        onActionSelected: (action) {
-          controller.handleEmailActionType(
-            action.action,
-            presentationEmail,
-            mailboxContain: mailboxContain,
-          );
+          if (shouldHandleAction(action.action)) {
+            controller.handleEmailActionType(
+              action.action,
+              presentationEmail,
+              mailboxContain: mailboxContain,
+            );
+          }
        },
      );

@@ -749,7 +774,7 @@ class ThreadView extends GetWidget<ThreadController>
        context,
        position,
        popupMenuWidget,
-      );
+      ).whenComplete(submenuController.hide);
    }
  }