TF-4227 Load all emails with tag when tag is selected

lib/features/mailbox/presentation/base_mailbox_view.dart

@@ -4,6 +4,7 @@ import 'package:core/presentation/views/list/tree_view.dart';
 import 'package:core/utils/platform_info.dart';
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
+import 'package:jmap_dart_client/jmap/core/id.dart';
 import 'package:model/extensions/presentation_mailbox_extension.dart';
 import 'package:model/extensions/session_extension.dart';
 import 'package:model/mailbox/expand_mode.dart';
@@ -396,6 +397,13 @@ abstract class BaseMailboxView extends GetWidget<MailboxController>
           .isLabelSettingEnabled
           .isTrue;
 
+      final selectedMailbox = controller.mailboxDashBoardController
+          .selectedMailbox.value;
+      Id? labelIdSelected;
+      if (selectedMailbox is PresentationLabelMailbox) {
+        labelIdSelected = selectedMailbox.id.id;
+      }
+
       if (isLabelCapabilitySupported && isLabelSettingEnabled) {
         final labelListExpandMode = labelController.labelListExpandMode.value;
         final labels = labelController.labels;
@@ -407,6 +415,7 @@ abstract class BaseMailboxView extends GetWidget<MailboxController>
                   labels: labels,
                   imagePaths: controller.imagePaths,
                   isDesktop: isDesktop,
+                  labelIdSelected: labelIdSelected,
                   onOpenLabelCallback: (label) => controller.openMailbox(
                     context,
                     PresentationLabelMailbox.initial(label),

lib/features/mailbox/presentation/extensions/presentation_mailbox_extension.dart

@@ -4,6 +4,7 @@ import 'package:flutter/material.dart';
 import 'package:jmap_dart_client/jmap/mail/email/keyword_identifier.dart';
 import 'package:model/extensions/presentation_mailbox_extension.dart';
 import 'package:model/mailbox/presentation_mailbox.dart';
+import 'package:tmail_ui_user/features/mailbox/presentation/model/presentation_label_mailbox.dart';
 import 'package:tmail_ui_user/main/localizations/app_localizations.dart';
 import 'package:tmail_ui_user/main/routes/app_routes.dart';
 import 'package:tmail_ui_user/main/routes/navigation_router.dart';
@@ -143,4 +144,18 @@ extension PresentationMailboxExtension on PresentationMailbox {
     }
     return null;
   }
+
+  bool get isCacheable => !isFavorite && this is! PresentationLabelMailbox;
+
+  bool get isLabelMailbox => this is PresentationLabelMailbox;
+
+  String? get filterKeyword {
+    if (isFavorite) {
+      return KeyWordIdentifier.emailFlagged.value;
+    }
+    if (this is PresentationLabelMailbox) {
+      return (this as PresentationLabelMailbox).label.keyword;
+    }
+    return null;
+  }
 }

lib/features/mailbox/presentation/widgets/labels/label_list_item.dart

@@ -1,3 +1,4 @@
+import 'package:core/presentation/extensions/color_extension.dart';
 import 'package:core/presentation/resources/image_paths.dart';
 import 'package:flutter/material.dart';
 import 'package:labels/extensions/label_extension.dart';
@@ -12,6 +13,7 @@ class LabelListItem extends StatelessWidget {
   final Label label;
   final ImagePaths imagePaths;
   final bool isDesktop;
+  final bool isSelected;
   final OnOpenLabelCallback onOpenLabelCallback;
 
   const LabelListItem({
@@ -20,6 +22,7 @@ class LabelListItem extends StatelessWidget {
     required this.imagePaths,
     required this.onOpenLabelCallback,
     this.isDesktop = false,
+    this.isSelected = false,
   });
 
   @override
@@ -38,7 +41,10 @@ class LabelListItem extends StatelessWidget {
         borderRadius: borderRadius,
         onTap: () => onOpenLabelCallback(label),
         child: Container(
-          decoration: BoxDecoration(borderRadius: borderRadius),
+          decoration: BoxDecoration(
+            borderRadius: borderRadius,
+            color: _backgroundColorItem,
+          ),
           padding: EdgeInsetsDirectional.symmetric(
             horizontal: isDesktop
                 ? MailboxItemWidgetStyles.itemPadding
@@ -70,4 +76,7 @@ class LabelListItem extends StatelessWidget {
       ),
     );
   }
+
+  Color get _backgroundColorItem =>
+      isSelected ? AppColor.blue100 : Colors.transparent;
 }

lib/features/mailbox/presentation/widgets/labels/label_list_view.dart

@@ -1,5 +1,6 @@
 import 'package:core/presentation/resources/image_paths.dart';
 import 'package:flutter/material.dart';
+import 'package:jmap_dart_client/jmap/core/id.dart';
 import 'package:labels/model/label.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/widgets/labels/label_list_item.dart';
 
@@ -7,6 +8,7 @@ class LabelListView extends StatelessWidget {
   final List<Label> labels;
   final ImagePaths imagePaths;
   final bool isDesktop;
+  final Id? labelIdSelected;
   final OnOpenLabelCallback onOpenLabelCallback;
 
   const LabelListView({
@@ -15,6 +17,7 @@ class LabelListView extends StatelessWidget {
     required this.imagePaths,
     required this.onOpenLabelCallback,
     this.isDesktop = false,
+    this.labelIdSelected,
   });
 
   @override
@@ -26,9 +29,11 @@ class LabelListView extends StatelessWidget {
       padding: EdgeInsets.zero,
       itemCount: labels.length,
       itemBuilder: (context, index) {
+        final label = labels[index];
         return LabelListItem(
-          label: labels[index],
+          label: label,
           imagePaths: imagePaths,
+          isSelected: label.id == labelIdSelected,
           isDesktop: isDesktop,
           onOpenLabelCallback: onOpenLabelCallback,
         );

lib/features/thread/presentation/filters/mailbox_filter_builder.dart

@@ -9,6 +9,7 @@ import 'package:model/email/presentation_email.dart';
 import 'package:model/extensions/keyword_identifier_extension.dart';
 import 'package:model/extensions/presentation_mailbox_extension.dart';
 import 'package:model/mailbox/presentation_mailbox.dart';
+import 'package:tmail_ui_user/features/mailbox/presentation/extensions/presentation_mailbox_extension.dart';
 import 'package:tmail_ui_user/features/thread/domain/model/email_filter.dart';
 import 'package:tmail_ui_user/features/thread/domain/model/filter_message_option.dart';
 
@@ -48,6 +49,13 @@ class MailboxFilterBuilder {
       return _buildActionRequiredMailboxFilter(before: before);
     }
 
+    if (selectedMailbox?.isLabelMailbox == true) {
+      return _buildKeywordBasedFilter(
+        selectedMailbox?.filterKeyword,
+        before: before,
+      );
+    }
+
     return buildDefaultMailboxFilter(before: before);
   }
 
@@ -110,6 +118,33 @@ class MailboxFilterBuilder {
     }
   }
 
+  Filter _buildKeywordBasedFilter({
+    required String keyword,
+    UTCDate? before,
+  }) {
+    switch (filterMessageOption) {
+      case FilterMessageOption.unread:
+        return EmailFilterCondition(
+          notKeyword: KeyWordIdentifier.emailSeen.value,
+          hasKeyword: keyword,
+          before: before,
+        );
+
+      case FilterMessageOption.attachments:
+        return EmailFilterCondition(
+          hasAttachment: true,
+          hasKeyword: keyword,
+          before: before,
+        );
+
+      default:
+        return EmailFilterCondition(
+          hasKeyword: keyword,
+          before: before,
+        );
+    }
+  }
+
   EmailFilterCondition buildDefaultMailboxFilter({UTCDate? before}) {
     switch (filterMessageOption) {
       case FilterMessageOption.all:
@@ -117,6 +152,7 @@ class MailboxFilterBuilder {
 
       case FilterMessageOption.unread:
         return EmailFilterCondition(
+          inMailbox: mailboxId,
           notKeyword: KeyWordIdentifier.emailSeen.value,
           before: before,
           inMailbox: _mailboxId,
@@ -124,6 +160,7 @@ class MailboxFilterBuilder {
 
       case FilterMessageOption.attachments:
         return EmailFilterCondition(
+          inMailbox: mailboxId,
           hasAttachment: true,
           before: before,
           inMailbox: _mailboxId,
@@ -131,6 +168,7 @@ class MailboxFilterBuilder {
 
       case FilterMessageOption.starred:
         return EmailFilterCondition(
+          inMailbox: mailboxId,
           hasKeyword: KeyWordIdentifier.emailFlagged.value,
           before: before,
           inMailbox: _mailboxId,

lib/features/thread/presentation/thread_controller.dart

@@ -26,6 +26,7 @@ import 'package:tmail_ui_user/features/email/presentation/model/composer_argumen
 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/domain/state/mark_as_mailbox_read_state.dart';
+import 'package:tmail_ui_user/features/mailbox/presentation/extensions/presentation_mailbox_extension.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/action/dashboard_action.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/controller/search_controller.dart' as search;
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/extensions/move_emails_to_mailbox_extension.dart';
@@ -63,6 +64,7 @@ import 'package:tmail_ui_user/features/thread/presentation/extensions/handle_ema
 import 'package:tmail_ui_user/features/thread/presentation/extensions/handle_keyboard_shortcut_actions_extension.dart';
 import 'package:tmail_ui_user/features/thread/presentation/extensions/list_presentation_email_extensions.dart';
 import 'package:tmail_ui_user/features/thread/presentation/extensions/refresh_thread_detail_extension.dart';
+import 'package:tmail_ui_user/features/thread/presentation/filters/mailbox_filter_builder.dart';
 import 'package:tmail_ui_user/features/thread/presentation/mixin/email_action_controller.dart';
 import 'package:tmail_ui_user/features/thread/presentation/model/delete_action_type.dart';
 import 'package:tmail_ui_user/features/thread/presentation/model/loading_more_status.dart';
@@ -592,7 +594,7 @@ class ThreadController extends BaseController with EmailActionController {
           _accountId!,
         ),
         getLatestChanges: getLatestChanges,
-        useCache: selectedMailbox?.isVirtualFolder != true,
+        useCache: selectedMailbox?.isCacheable ?? false,
       ));
     } else {
       consumeState(Stream.value(Left(GetAllEmailFailure(NotFoundSessionException()))));