From 22072bdfa52a781ed26f72a63e88a83eea42f43c Mon Sep 17 00:00:00 2001
From: HuyNguyen <quanghnguyen@linagora.com>
Date: Fri, 24 Feb 2023 16:03:21 +0700
Subject: [PATCH] TF-1478: Update menu option for other mailbox

---
 .../presentation/mixin/mailbox_widget_mixin.dart       | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/lib/features/mailbox/presentation/mixin/mailbox_widget_mixin.dart b/lib/features/mailbox/presentation/mixin/mailbox_widget_mixin.dart
index 6f11f9e57..4e6c81d9f 100644
--- a/lib/features/mailbox/presentation/mixin/mailbox_widget_mixin.dart
+++ b/lib/features/mailbox/presentation/mixin/mailbox_widget_mixin.dart
@@ -31,12 +31,15 @@ mixin MailboxWidgetMixin {
     PresentationMailbox mailbox,
     bool spamReportEnabled
   ) {
+    final _notSupportShowMarkAsRead = mailbox.isDrafts || mailbox.isSent || mailbox.isTrash;
+
     return [
       if (BuildUtils.isWeb)
         MailboxActions.openInNewTab,
       if (mailbox.isSpam)
         _mailboxActionForSpam(spamReportEnabled),
-      MailboxActions.markAsRead
+      if (mailbox.getCountUnReadEmails().isNotEmpty && !_notSupportShowMarkAsRead)
+        MailboxActions.markAsRead
     ];
   }
 
@@ -44,7 +47,8 @@ mixin MailboxWidgetMixin {
     return [
       if (BuildUtils.isWeb && mailbox.isSubscribedMailbox)
         MailboxActions.openInNewTab,
-      MailboxActions.markAsRead,
+      if (mailbox.getCountUnReadEmails().isNotEmpty)
+        MailboxActions.markAsRead,
       MailboxActions.move,
       MailboxActions.rename,
       MailboxActions.delete,
@@ -59,7 +63,7 @@ mixin MailboxWidgetMixin {
     return [
       if (BuildUtils.isWeb && mailbox.isSubscribedMailbox)
         MailboxActions.openInNewTab,
-      if (mailbox.isChildOfTeamMailboxes)
+      if (mailbox.isChildOfTeamMailboxes && mailbox.getCountUnReadEmails().isNotEmpty)
         MailboxActions.markAsRead,
       if (mailbox.isTeamMailboxes)
         if (mailbox.isSubscribedMailbox)