diff --git a/lib/features/base/widget/popup_menu/popup_submenu_controller.dart b/lib/features/base/widget/popup_menu/popup_submenu_controller.dart index 24261dddd..101092667 100644 --- a/lib/features/base/widget/popup_menu/popup_submenu_controller.dart +++ b/lib/features/base/widget/popup_menu/popup_submenu_controller.dart @@ -44,16 +44,16 @@ class PopupSubmenuController { _submenuEntry = OverlayEntry( builder: (_) { - return Positioned( - left: finalLeft, + return PositionedDirectional( + start: finalLeft, top: anchor.top, child: MouseRegion( onExit: (_) => hide(), child: Material( elevation: 8, color: Colors.white, - shape: RoundedRectangleBorder( - borderRadius: BorderRadius.circular(6), + shape: const RoundedRectangleBorder( + borderRadius: BorderRadius.all(Radius.circular(6)), ), child: SizedBox( width: submenuWidth, @@ -66,7 +66,7 @@ class PopupSubmenuController { }, ); - Overlay.of(context).insert(_submenuEntry!); + Overlay.maybeOf(context)?.insert(_submenuEntry!); } void hide() { diff --git a/lib/features/email/presentation/controller/single_email_controller.dart b/lib/features/email/presentation/controller/single_email_controller.dart index ec1378ad0..6368f8160 100644 --- a/lib/features/email/presentation/controller/single_email_controller.dart +++ b/lib/features/email/presentation/controller/single_email_controller.dart @@ -35,6 +35,7 @@ import 'package:tmail_ui_user/features/email/domain/model/event_action.dart'; import 'package:tmail_ui_user/features/email/domain/model/mark_read_action.dart'; import 'package:tmail_ui_user/features/email/domain/model/send_receipt_to_sender_request.dart'; import 'package:tmail_ui_user/features/email/domain/model/view_entire_message_request.dart'; +import 'package:tmail_ui_user/features/email/domain/state/add_a_label_to_an_email_state.dart'; import 'package:tmail_ui_user/features/email/domain/state/calendar_event_accept_state.dart'; import 'package:tmail_ui_user/features/email/domain/state/calendar_event_counter_accept_state.dart'; import 'package:tmail_ui_user/features/email/domain/state/calendar_event_maybe_state.dart'; @@ -48,6 +49,7 @@ import 'package:tmail_ui_user/features/email/domain/state/parse_calendar_event_s import 'package:tmail_ui_user/features/email/domain/state/print_email_state.dart'; import 'package:tmail_ui_user/features/email/domain/state/send_receipt_to_sender_state.dart'; import 'package:tmail_ui_user/features/email/domain/state/unsubscribe_email_state.dart'; +import 'package:tmail_ui_user/features/email/domain/usecases/add_a_label_to_an_email_interactor.dart'; import 'package:tmail_ui_user/features/email/domain/usecases/calendar_event_accept_interactor.dart'; import 'package:tmail_ui_user/features/email/domain/usecases/calendar_event_counter_accept_interactor.dart'; import 'package:tmail_ui_user/features/email/domain/usecases/calendar_event_reject_interactor.dart'; @@ -65,6 +67,7 @@ import 'package:tmail_ui_user/features/email/presentation/bindings/calendar_even import 'package:tmail_ui_user/features/email/presentation/bindings/mdn_interactor_bindings.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/calendar_attendee_extension.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/calendar_organizer_extension.dart'; +import 'package:tmail_ui_user/features/email/presentation/extensions/handle_label_for_email_extension.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/handle_mail_action_by_shortcut_action_extension.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/handle_open_attachment_list_extension.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/update_attendance_status_extension.dart'; @@ -125,6 +128,7 @@ class SingleEmailController extends BaseController with AppLoaderMixin { RejectCalendarEventInteractor? _rejectCalendarEventInteractor; AcceptCounterCalendarEventInteractor? _acceptCounterCalendarEventInteractor; ThreadDetailController? _threadDetailController; + AddALabelToAnEmailInteractor? addALabelToAnEmailInteractor; final emailContents = RxnString(); final attachments = [].obs; @@ -240,6 +244,8 @@ class SingleEmailController extends BaseController with AppLoaderMixin { _handlePrintEmailSuccess(success); } else if (success is CalendarEventReplySuccess) { calendarEventSuccess(success); + } else if (success is AddALabelToAnEmailSuccess) { + handleAddLabelToEmailSuccess(success); } else { super.handleSuccessViewState(success); } @@ -257,6 +263,8 @@ class SingleEmailController extends BaseController with AppLoaderMixin { _showMessageWhenEmailPrintingFailed(failure); } else if (failure is CalendarEventReplyFailure) { _calendarEventFailure(failure); + } else if (failure is AddALabelToAnEmailFailure) { + handleAddLabelToEmailFailure(failure); } else { super.handleFailureViewState(failure); } @@ -1551,9 +1559,4 @@ class SingleEmailController extends BaseController with AppLoaderMixin { _threadDetailController?.onNextMobile(); } } - - bool get isLabelFeatureEnabled { - return mailboxDashBoardController.isLabelCapabilitySupported && - mailboxDashBoardController.labelController.isLabelSettingEnabled.isTrue; - } } \ No newline at end of file diff --git a/lib/features/email/presentation/email_view.dart b/lib/features/email/presentation/email_view.dart index f82303bab..aba36daff 100644 --- a/lib/features/email/presentation/email_view.dart +++ b/lib/features/email/presentation/email_view.dart @@ -22,6 +22,7 @@ import 'package:tmail_ui_user/features/base/widget/optional_expanded.dart'; import 'package:tmail_ui_user/features/base/widget/optional_scroll.dart'; import 'package:tmail_ui_user/features/email/presentation/controller/single_email_controller.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/calendar_event_extension.dart'; +import 'package:tmail_ui_user/features/email/presentation/extensions/handle_label_for_email_extension.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/handle_on_iframe_click_in_email_extension.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/presentation_email_extension.dart'; import 'package:tmail_ui_user/features/email/presentation/extensions/validate_display_free_busy_message_extension.dart'; @@ -106,6 +107,12 @@ class EmailView extends GetWidget { labels: controller.mailboxDashBoardController.labelController.labels, openBottomSheetContextMenu: controller.mailboxDashBoardController.openBottomSheetContextMenu, openPopupMenu: controller.mailboxDashBoardController.openPopupMenuActionGroup, + onSelectLabelAction: (label, isSelected) => + controller.toggleLabelToEmail( + presentationEmail.id!, + label, + isSelected, + ), ); }, supportBackAction: !isInsideThreadDetailView, @@ -313,6 +320,12 @@ class EmailView extends GetWidget { labels: controller.mailboxDashBoardController.labelController.labels, openBottomSheetContextMenu: controller.mailboxDashBoardController.openBottomSheetContextMenu, openPopupMenu: controller.mailboxDashBoardController.openPopupMenuActionGroup, + onSelectLabelAction: (label, isSelected) => + controller.toggleLabelToEmail( + presentationEmail.id!, + label, + isSelected, + ), ), onToggleThreadDetailCollapseExpand: onToggleThreadDetailCollapseExpand, mailboxContain: presentationEmail.findMailboxContain( diff --git a/lib/features/email/presentation/extensions/email_loaded_extension.dart b/lib/features/email/presentation/extensions/email_loaded_extension.dart new file mode 100644 index 000000000..033d82d38 --- /dev/null +++ b/lib/features/email/presentation/extensions/email_loaded_extension.dart @@ -0,0 +1,31 @@ +import 'package:jmap_dart_client/jmap/mail/email/email.dart'; +import 'package:jmap_dart_client/jmap/mail/email/keyword_identifier.dart'; +import 'package:model/extensions/email_extension.dart'; +import 'package:tmail_ui_user/features/email/presentation/model/email_loaded.dart'; +import 'package:tmail_ui_user/features/thread/data/extensions/map_keywords_extension.dart'; + +extension EmailLoadedExtension on EmailLoaded { + EmailLoaded addEmailKeyword({ + required EmailId emailId, + required KeyWordIdentifier keyword, + }) { + if (emailCurrent == null || emailCurrent?.id != emailId) { + return this; + } + final newKeyword = emailCurrent?.keywords?.withKeyword(keyword); + final updatedEmail = emailCurrent?.copyWith(keywords: newKeyword); + return copyWith(emailCurrent: updatedEmail); + } + + EmailLoaded removeEmailKeyword({ + required EmailId emailId, + required KeyWordIdentifier keyword, + }) { + if (emailCurrent == null || emailCurrent?.id != emailId) { + return this; + } + final newKeyword = emailCurrent?.keywords?.withoutKeyword(keyword); + final updatedEmail = emailCurrent?.copyWith(keywords: newKeyword); + return copyWith(emailCurrent: updatedEmail); + } +} diff --git a/lib/features/email/presentation/extensions/handle_label_for_email_extension.dart b/lib/features/email/presentation/extensions/handle_label_for_email_extension.dart new file mode 100644 index 000000000..6f6b190fb --- /dev/null +++ b/lib/features/email/presentation/extensions/handle_label_for_email_extension.dart @@ -0,0 +1,187 @@ +import 'package:core/utils/platform_info.dart'; +import 'package:dartz/dartz.dart'; +import 'package:get/get.dart'; +import 'package:jmap_dart_client/jmap/account_id.dart'; +import 'package:jmap_dart_client/jmap/mail/email/email.dart'; +import 'package:jmap_dart_client/jmap/mail/email/keyword_identifier.dart'; +import 'package:labels/extensions/label_extension.dart'; +import 'package:labels/model/label.dart'; +import 'package:tmail_ui_user/features/email/domain/state/add_a_label_to_an_email_state.dart'; +import 'package:tmail_ui_user/features/email/domain/usecases/add_a_label_to_an_email_interactor.dart'; +import 'package:tmail_ui_user/features/email/presentation/controller/single_email_controller.dart'; +import 'package:tmail_ui_user/features/email/presentation/extensions/email_loaded_extension.dart'; +import 'package:tmail_ui_user/features/email/presentation/extensions/presentation_email_extension.dart'; +import 'package:tmail_ui_user/features/home/data/exceptions/session_exceptions.dart'; +import 'package:tmail_ui_user/features/labels/domain/exceptions/label_exceptions.dart'; +import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/extensions/update_current_emails_flags_extension.dart'; +import 'package:tmail_ui_user/features/thread/domain/extensions/presentation_email_map_extension.dart'; +import 'package:tmail_ui_user/features/thread_detail/domain/extensions/list_email_in_thread_detail_info_extension.dart'; +import 'package:tmail_ui_user/main/routes/route_navigation.dart'; + +extension HandleLabelForEmailExtension on SingleEmailController { + bool get isLabelFeatureEnabled { + return mailboxDashBoardController.isLabelCapabilitySupported && + mailboxDashBoardController.labelController.isLabelSettingEnabled.isTrue; + } + + void toggleLabelToEmail(EmailId emailId, Label label, bool isSelected) { + if (isSelected) { + final accountId = mailboxDashBoardController.accountId.value; + + _addALabelToAnEmail( + accountId: accountId, + emailId: emailId, + label: label, + ); + } + } + + void _addALabelToAnEmail({ + required AccountId? accountId, + required Label label, + required EmailId emailId, + }) { + final labelDisplay = label.safeDisplayName; + + if (accountId == null) { + consumeState( + Stream.value( + Left(AddALabelToAnEmailFailure( + exception: NotFoundAccountIdException(), + labelDisplay: labelDisplay, + )), + ), + ); + return; + } + + final labelKeyword = label.keyword; + if (labelKeyword == null) { + consumeState( + Stream.value(Left( + AddALabelToAnEmailFailure( + exception: LabelKeywordIsNull(), + labelDisplay: labelDisplay, + ), + )), + ); + return; + } + + addALabelToAnEmailInteractor = getBinding(); + if (addALabelToAnEmailInteractor != null) { + consumeState(addALabelToAnEmailInteractor!.execute( + accountId, + emailId, + labelKeyword, + label.safeDisplayName, + )); + } else { + consumeState( + Stream.value( + Left(AddALabelToAnEmailFailure( + exception: LabelInteractorIsNull(), + labelDisplay: labelDisplay, + )), + ), + ); + } + } + + void handleAddLabelToEmailSuccess(AddALabelToAnEmailSuccess success) { + toastManager.showMessageSuccess(success); + + _autoSyncLabelToSelectedEmailOnMemory( + emailId: success.emailId, + labelKeyword: success.labelKeyword, + ); + } + + void handleAddLabelToEmailFailure(AddALabelToAnEmailFailure failure) { + toastManager.showMessageFailure(failure); + } + + void _autoSyncLabelToSelectedEmailOnMemory({ + required EmailId emailId, + required KeyWordIdentifier labelKeyword, + }) { + if (PlatformInfo.isMobile && !isThreadDetailEnabled) { + _updateLabelInEmailWithThreadDisabledOnMobile( + emailId: emailId, + labelKeyword: labelKeyword, + ); + } else { + _updateLabelInEmailWithThreadEnabled( + emailId: emailId, + labelKeyword: labelKeyword, + ); + } + } + + void _updateLabelInEmailWithThreadDisabledOnMobile({ + required EmailId emailId, + required KeyWordIdentifier labelKeyword, + }) { + final selectedEmail = mailboxDashBoardController.selectedEmail.value; + final selectedEmailId = selectedEmail?.id; + if (selectedEmail == null || + selectedEmailId == null || + selectedEmailId != emailId) { + return; + } + + mailboxDashBoardController.selectedEmail.value = + selectedEmail.addKeyword(labelKeyword); + + final emailLoaded = currentEmailLoaded.value; + if (emailLoaded != null) { + currentEmailLoaded.value = emailLoaded.addEmailKeyword( + emailId: emailId, + keyword: labelKeyword, + ); + } + + mailboxDashBoardController.updateEmailFlagByEmailIds( + [emailId], + isLabelAdded: true, + labelKeyword: labelKeyword, + ); + } + + void _updateLabelInEmailWithThreadEnabled({ + required EmailId emailId, + required KeyWordIdentifier labelKeyword, + }) { + final controller = threadDetailController; + if (controller == null) return; + + final currentEmailId = currentEmail?.id; + if (currentEmailId == null || currentEmailId != emailId) return; + + controller.emailIdsPresentation.value = + controller.emailIdsPresentation.addEmailKeywordById( + emailId: emailId, + keyword: labelKeyword, + ); + + controller.emailsInThreadDetailInfo.value = + controller.emailsInThreadDetailInfo.addEmailKeywordById( + emailId: emailId, + keyword: labelKeyword, + ); + + final emailLoaded = controller.currentEmailLoaded.value; + if (emailLoaded != null) { + controller.currentEmailLoaded.value = emailLoaded.addEmailKeyword( + emailId: emailId, + keyword: labelKeyword, + ); + } + + mailboxDashBoardController.updateEmailFlagByEmailIds( + [emailId], + isLabelAdded: true, + labelKeyword: labelKeyword, + ); + } +} diff --git a/lib/features/email/presentation/extensions/presentation_email_extension.dart b/lib/features/email/presentation/extensions/presentation_email_extension.dart index 240adfaf1..daad75efe 100644 --- a/lib/features/email/presentation/extensions/presentation_email_extension.dart +++ b/lib/features/email/presentation/extensions/presentation_email_extension.dart @@ -1,4 +1,5 @@ import 'package:jmap_dart_client/jmap/mail/email/email_address.dart'; +import 'package:jmap_dart_client/jmap/mail/email/keyword_identifier.dart'; import 'package:labels/model/label.dart'; import 'package:model/email/email_action_type.dart'; import 'package:model/email/presentation_email.dart'; @@ -168,4 +169,16 @@ extension PresentationEmailExtension on PresentationEmail { label.keyword != null && enabledKeywords.contains(label.keyword)) .toList(); } + + PresentationEmail addKeyword(KeyWordIdentifier keyword) { + return copyWith( + keywords: keywords.withKeyword(keyword), + ); + } + + PresentationEmail removeKeyword(KeyWordIdentifier keyword) { + return copyWith( + keywords: keywords.withoutKeyword(keyword), + ); + } } diff --git a/lib/features/email/presentation/model/email_loaded.dart b/lib/features/email/presentation/model/email_loaded.dart index 15d942d79..0d0175be9 100644 --- a/lib/features/email/presentation/model/email_loaded.dart +++ b/lib/features/email/presentation/model/email_loaded.dart @@ -17,6 +17,20 @@ class EmailLoaded with EquatableMixin { this.emailCurrent, }); + EmailLoaded copyWith({ + String? htmlContent, + List? attachments, + List? inlineImages, + Email? emailCurrent, + }) { + return EmailLoaded( + htmlContent: htmlContent ?? this.htmlContent, + attachments: attachments ?? this.attachments, + inlineImages: inlineImages ?? this.inlineImages, + emailCurrent: emailCurrent ?? this.emailCurrent, + ); + } + SMimeSignatureStatus? get sMimeStatus => emailCurrent?.sMimeStatus; @override diff --git a/lib/features/email/presentation/utils/email_action_reactor/email_action_reactor.dart b/lib/features/email/presentation/utils/email_action_reactor/email_action_reactor.dart index b3d8696a1..7a3c79e03 100644 --- a/lib/features/email/presentation/utils/email_action_reactor/email_action_reactor.dart +++ b/lib/features/email/presentation/utils/email_action_reactor/email_action_reactor.dart @@ -52,6 +52,7 @@ 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/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'; @@ -490,6 +491,7 @@ class EmailActionReactor { required OpenBottomSheetContextMenuAction openBottomSheetContextMenu, required OpenPopupMenuActionGroup openPopupMenu, List