TF-1176 Fix delete button is enable even when no mailbox is selected

core/lib/presentation/views/button/button_builder.dart

@@ -25,6 +25,7 @@ class ButtonBuilder {
   double? _radiusSplash;
   double? _maxWidth;
   EdgeInsets? _padding;
+  String? _tooltip;
 
   void key(Key key) {
     _key = key;
@@ -79,6 +80,10 @@ class ButtonBuilder {
     _isVertical = isVertical;
   }
 
+  void tooltip(String? message) {
+    _tooltip = message;
+  }
+
   ButtonBuilder(this._icon);
 
   void onPressActionClick(OnPressActionClick onPressActionClick) {
@@ -109,14 +114,17 @@ class ButtonBuilder {
         },
         customBorder: RoundedRectangleBorder(
             borderRadius: BorderRadius.circular(_radiusSplash ?? 20)),
-        child: Container(
-          key: _key,
-          alignment: Alignment.center,
-          color: _decoration == null ? _colorButton : null,
-          decoration: _decoration,
-          width: _maxWidth,
-          padding: _padding ?? EdgeInsets.zero,
-          child: _buildBody()
+        child: Tooltip(
+          message: _tooltip ?? '',
+          child: Container(
+            key: _key,
+            alignment: Alignment.center,
+            color: _decoration == null ? _colorButton : null,
+            decoration: _decoration,
+            width: _maxWidth,
+            padding: _padding ?? EdgeInsets.zero,
+            child: _buildBody()
+          ),
         )
       ),
     );

lib/features/mailbox/presentation/mailbox_controller.dart

@@ -573,6 +573,31 @@ class MailboxController extends BaseMailboxController with MailboxActionHandlerM
 
   bool isSelectionEnabled() => currentSelectMode.value == SelectMode.ACTIVE;
 
+  List<MailboxActions> get listActionOfMailboxSelected {
+    final currentMailboxesSelected = listMailboxSelected;
+
+    if (currentMailboxesSelected.length == 1) {
+      if (currentMailboxesSelected.isAllDefaultMailboxes && currentMailboxesSelected.isAllUnreadMailboxes) {
+        return [MailboxActions.markAsRead];
+      } else if (currentMailboxesSelected.isAllPersonalMailboxes) {
+        return [
+          MailboxActions.move,
+          MailboxActions.rename,
+          if (currentMailboxesSelected.isAllUnreadMailboxes)
+            MailboxActions.markAsRead,
+          MailboxActions.delete
+        ];
+      } else {
+        return [];
+      }
+    } else if (currentMailboxesSelected.length > 1
+      && currentMailboxesSelected.isAllPersonalMailboxes) {
+      return [MailboxActions.delete];
+    } else {
+      return [];
+    }
+  }
+
   void _cancelSelectMailbox() {
     unAllSelectedMailboxNode();
     currentSelectMode.value = SelectMode.INACTIVE;

lib/features/mailbox/presentation/mailbox_view.dart

@@ -58,9 +58,14 @@ class MailboxView extends GetWidget<MailboxController>
                                 )
                               ),
                             )),
-                            Obx(() => controller.isSelectionEnabled()
-                                ? _buildOptionSelectionMailbox(context)
-                                : const SizedBox.shrink()),
+                            Obx(() {
+                              if (controller.isSelectionEnabled()
+                                  && controller.listActionOfMailboxSelected.isNotEmpty) {
+                                return _buildOptionSelectionMailbox(context);
+                              } else {
+                                return const SizedBox.shrink();
+                              }
+                            }),
                           ]),
                         ),
                         Obx(() => controller.isMailboxListScrollable.isTrue
@@ -377,21 +382,32 @@ class MailboxView extends GetWidget<MailboxController>
   }
 
   Widget _buildOptionSelectionMailbox(BuildContext context) {
-    return Column(children: [
-      const Divider(color: AppColor.lineItemListColor, height: 1, thickness: 0.2),
-      SafeArea(
-        child: Padding(
-          padding: const EdgeInsets.symmetric(vertical: 8),
-          child: (BottomBarSelectionMailboxWidget(context,
-                  _imagePaths,
-                  controller.listMailboxSelected)
-              ..addOnMailboxActionsClick((actions, listMailboxSelected) =>
-                  controller.pressMailboxSelectionAction(
-                      context,
-                      actions,
-                      listMailboxSelected)))
-            .build()))
-    ]);
+    return Column(
+      mainAxisSize: MainAxisSize.min,
+      children: [
+        const Divider(
+          color: AppColor.lineItemListColor,
+          height: 1,
+          thickness: 0.2
+        ),
+        SafeArea(
+          right: false,
+          top: false,
+          child: BottomBarSelectionMailboxWidget(
+            _responsiveUtils,
+            _imagePaths,
+            controller.listMailboxSelected,
+            controller.listActionOfMailboxSelected,
+            onMailboxActionsClick: (actions, listMailboxSelected) =>
+              controller.pressMailboxSelectionAction(
+                context,
+                actions,
+                listMailboxSelected
+              )
+          )
+        )
+      ]
+    );
   }
 
   Widget _buildVersionInformation(BuildContext context, PackageInfo packageInfo) {

lib/features/mailbox/presentation/widgets/bottom_bar_selection_mailbox_widget.dart

@@ -1,133 +1,53 @@
-import 'package:core/core.dart';
+import 'package:core/presentation/extensions/color_extension.dart';
+import 'package:core/presentation/resources/image_paths.dart';
+import 'package:core/presentation/utils/responsive_utils.dart';
+import 'package:core/presentation/views/button/button_builder.dart';
 import 'package:flutter/material.dart';
-import 'package:model/model.dart';
+import 'package:model/mailbox/presentation_mailbox.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/model/mailbox_actions.dart';
-import 'package:tmail_ui_user/main/localizations/app_localizations.dart';
 
 typedef OnMailboxActionsClick = void Function(MailboxActions, List<PresentationMailbox>);
 
-class BottomBarSelectionMailboxWidget {
+class BottomBarSelectionMailboxWidget extends StatelessWidget {
 
-  final BuildContext _context;
+  final ResponsiveUtils _responsiveUtils;
   final ImagePaths _imagePaths;
   final List<PresentationMailbox> _listSelectionMailbox;
+  final List<MailboxActions> _listMailboxActions;
+  final OnMailboxActionsClick onMailboxActionsClick;
 
-  OnMailboxActionsClick? _onMailboxActionsClick;
-
-  BottomBarSelectionMailboxWidget(
-    this._context,
+  const BottomBarSelectionMailboxWidget(
+    this._responsiveUtils,
     this._imagePaths,
     this._listSelectionMailbox,
-  );
+    this._listMailboxActions,
+    {
+      Key? key,
+      required this.onMailboxActionsClick
+    }
+  ) : super(key: key);
 
-  void addOnMailboxActionsClick(OnMailboxActionsClick onMailboxActionsClick) {
-    _onMailboxActionsClick = onMailboxActionsClick;
+
+  @override
+  Widget build(BuildContext context) {
+    return Row(children: _listMailboxActions
+      .map((action) => _buildMailboxActionButton(context, action))
+      .toList());
   }
 
-  Widget build() {
-    return Container(
-      key: const Key('bottom_bar_selection_mailbox_widget'),
-      alignment: Alignment.center,
-      color: Colors.white,
-      child: MediaQuery(
-        data: const MediaQueryData(padding: EdgeInsets.zero),
-        child: SafeArea(child: _buildListOptionButton())
+  Widget _buildMailboxActionButton(BuildContext context, MailboxActions actions) {
+    return Expanded(child: (ButtonBuilder(actions.getContextMenuIcon(_imagePaths))
+      ..key(const Key('button_move_all_mailbox'))
+      ..radiusSplash(8)
+      ..padding(const EdgeInsets.all(8))
+      ..tooltip(actions.getTitleContextMenu(context))
+      ..textStyle(const TextStyle(fontSize: 12, color: AppColor.colorTextButton))
+      ..iconColor(AppColor.colorTextButton)
+      ..onPressActionClick(() => onMailboxActionsClick.call(actions, _listSelectionMailbox))
+      ..text(
+        _responsiveUtils.isLandscapeMobile(context) ? null : actions.getTitleContextMenu(context),
+        isVertical: true
       )
-    );
+    ).build());
   }
-
-  Widget _buildListOptionButton() {
-    return Row(
-      mainAxisAlignment: MainAxisAlignment.center,
-      crossAxisAlignment: CrossAxisAlignment.center,
-      children: [
-        Expanded(child: (ButtonBuilder(_imagePaths.icMoveMailbox)
-            ..key(const Key('button_move_all_mailbox'))
-            ..paddingIcon(const EdgeInsets.all(8))
-            ..textStyle(TextStyle(
-                fontSize: 12,
-                color: _isMoveMailboxValid
-                    ? AppColor.colorTextButton
-                    : AppColor.colorTextButton.withOpacity(0.3)))
-            ..iconColor(_isMoveMailboxValid
-                ? AppColor.colorTextButton
-                : AppColor.colorTextButton.withOpacity(0.3))
-            ..onPressActionClick(() {
-              if (_isMoveMailboxValid) {
-                _onMailboxActionsClick?.call(MailboxActions.move, _listSelectionMailbox);
-              }
-            })
-            ..text(AppLocalizations.of(_context).move, isVertical: true))
-          .build()),
-        Expanded(child: (ButtonBuilder(_imagePaths.icRenameMailbox)
-            ..key(const Key('button_rename_mailbox'))
-            ..paddingIcon(const EdgeInsets.all(8))
-            ..textStyle(TextStyle(
-                fontSize: 12,
-                color: _isRenameMailboxValid
-                    ? AppColor.colorTextButton
-                    : AppColor.colorTextButton.withOpacity(0.3)))
-            ..iconColor(_isRenameMailboxValid
-                ? AppColor.colorTextButton
-                : AppColor.colorTextButton.withOpacity(0.3))
-            ..onPressActionClick(() {
-              if (_isRenameMailboxValid) {
-                _onMailboxActionsClick?.call(MailboxActions.rename, _listSelectionMailbox);
-              }
-            })
-            ..text(AppLocalizations.of(_context).rename, isVertical: true))
-          .build()),
-        Expanded(child: (ButtonBuilder(_imagePaths.icMarkAsRead)
-            ..key(const Key('button_mark_read_all_mailbox'))
-            ..paddingIcon(const EdgeInsets.all(8))
-            ..textStyle(TextStyle(
-                fontSize: 12,
-                color: _isMarkAsReadMailboxValid
-                    ? AppColor.colorTextButton
-                    : AppColor.colorTextButton.withOpacity(0.3)))
-            ..iconColor(_isMarkAsReadMailboxValid
-                ? AppColor.colorTextButton
-                : AppColor.colorTextButton.withOpacity(0.3))
-            ..onPressActionClick(() {
-              if (_isMarkAsReadMailboxValid) {
-                _onMailboxActionsClick?.call(MailboxActions.markAsRead, _listSelectionMailbox);
-              }
-            })
-            ..text(AppLocalizations.of(_context).mark_as_read, isVertical: true))
-          .build()),
-        Expanded(child: (ButtonBuilder(_imagePaths.icDeleteMailbox)
-            ..key(const Key('button_delete_all_mailbox'))
-            ..paddingIcon(const EdgeInsets.all(8))
-            ..textStyle(TextStyle(
-                fontSize: 12,
-                color: _isDeleteMailboxValid
-                    ? AppColor.colorTextButton
-                    : AppColor.colorTextButton.withOpacity(0.3)))
-            ..iconColor(_isDeleteMailboxValid
-                ? AppColor.colorTextButton
-                : AppColor.colorTextButton.withOpacity(0.3))
-            ..onPressActionClick(() {
-              if (_isDeleteMailboxValid) {
-                _onMailboxActionsClick?.call(MailboxActions.delete, _listSelectionMailbox);
-              }
-            })
-            ..text(AppLocalizations.of(_context).delete, isVertical: true))
-          .build())
-      ]
-    );
-  }
-
-  bool get _isDeleteMailboxValid => _isAllFolderMailbox;
-
-  bool get _isRenameMailboxValid => _listSelectionMailbox.length == 1
-      && _isAllFolderMailbox;
-
-  bool get _isMarkAsReadMailboxValid => _listSelectionMailbox.length == 1
-      && _listSelectionMailbox.first.getCountUnReadEmails().isNotEmpty;
-
-  bool get _isMoveMailboxValid => _listSelectionMailbox.length == 1
-      && _isAllFolderMailbox;
-
-  bool get _isAllFolderMailbox =>
-      _listSelectionMailbox.every((mailbox) => !mailbox.hasRole());
 }

model/lib/extensions/list_presentation_mailbox_extension.dart

@@ -11,4 +11,10 @@ extension ListPresentationMailboxExtension on List<PresentationMailbox> {
 
   List<PresentationMailbox> get listPersonalMailboxes =>
     where((mailbox) => mailbox.isPersonal).toList();
+
+  bool get isAllPersonalMailboxes => every((mailbox) => mailbox.isPersonal && !mailbox.isDefault);
+
+  bool get isAllDefaultMailboxes => every((mailbox) => mailbox.isDefault);
+
+  bool get isAllUnreadMailboxes => every((mailbox) => mailbox.getCountUnReadEmails().isNotEmpty);
 }