TF-3996 Group context menu actions when opened email

This commit is contained in:
dab246
2025-09-03 12:31:47 +07:00
committed by Dat H. Pham
parent 22379dbbbd
commit b66d84ecb8
10 changed files with 168 additions and 10 deletions
@@ -0,0 +1,44 @@
extension IterableExtension<T> on Iterable<T> {
/// Group elements by [keySelector].
/// If [sortKeys] = true, the keys will be sorted in ascending order,
/// key == -1 will always be at the bottom (applies to int keys).
Map<K, List<T>> groupBy<K>(
K Function(T element) keySelector, {
bool sortKeys = false,
}) {
final map = <K, List<T>>{};
for (final element in this) {
final key = keySelector(element);
map.putIfAbsent(key, () => []).add(element);
}
Iterable<K> sortedKeys;
if (sortKeys && K == int) {
sortedKeys = map.keys.toList()
..sort((a, b) {
final ai = a as int;
final bi = b as int;
if (ai == -1 && bi != -1) return 1;
if (bi == -1 && ai != -1) return -1;
return ai.compareTo(bi);
});
} else if (sortKeys) {
sortedKeys = map.keys.toList()
..sort((a, b) {
if (a is Comparable && b is Comparable) {
return (a as Comparable).compareTo(b);
}
return 0;
});
} else {
sortedKeys = map.keys;
}
final result = <K, List<T>>{};
for (final key in sortedKeys) {
result[key] = map[key]!;
}
return result;
}
}
@@ -0,0 +1,57 @@
import 'package:core/presentation/extensions/iterable_extension.dart';
import 'package:flutter_test/flutter_test.dart';
void main() {
group('IterableExtension', () {
group('groupBy method', () {
test('group by int keys without sort', () {
final data = [1, 2, 3, 4];
final result = data.groupBy((e) => e % 2);
expect(result.keys.toList(), [1, 0]);
expect(result[0], [2, 4]);
expect(result[1], [1, 3]);
});
test('group by int keys with sort, -1 goes last', () {
final data = [10, 20, 30, 40];
final categories = [2, -1, 1, 3];
final result = data.groupBy((e) {
final idx = data.indexOf(e);
return categories[idx];
}, sortKeys: true);
expect(result.keys.toList(), [1, 2, 3, -1]);
expect(result[1], [30]);
expect(result[2], [10]);
expect(result[3], [40]);
expect(result[-1], [20]);
});
test('group by string keys with sort', () {
final words = ['apple', 'banana', 'cherry', 'avocado'];
final result = words.groupBy((w) => w[0], sortKeys: true);
expect(result.keys.toList(), ['a', 'b', 'c']);
expect(result['a'], ['apple', 'avocado']);
expect(result['b'], ['banana']);
expect(result['c'], ['cherry']);
});
test('group with multiple elements in same group', () {
final data = ['cat', 'car', 'dog', 'door'];
final result = data.groupBy((s) => s[0]);
expect(result['c'], ['cat', 'car']);
expect(result['d'], ['dog', 'door']);
});
test('empty list returns empty map', () {
final data = <int>[];
final result = data.groupBy((e) => e);
expect(result, isEmpty);
});
});
});
}
@@ -0,0 +1,8 @@
import 'package:core/presentation/extensions/iterable_extension.dart';
import 'package:tmail_ui_user/features/base/widget/context_menu/context_menu_item_action.dart';
extension ContextMenuActionListExt on List<ContextMenuItemAction> {
Map<int, List<ContextMenuItemAction>> groupByCategory() {
return groupBy<int>((action) => action.category, sortKeys: true);
}
}
@@ -27,6 +27,7 @@ mixin PopupContextMenuActionMixin {
required List<ContextMenuItemAction> itemActions,
required OnContextMenuActionClick onContextMenuActionClick,
Key? key,
bool useGroupedActions = false,
}) async {
return await showModalBottomSheet(
context: context,
@@ -50,6 +51,7 @@ mixin PopupContextMenuActionMixin {
child: ContextMenuDialogView(
actions: itemActions,
onContextMenuActionClick: onContextMenuActionClick,
useGroupedActions: useGroupedActions,
),
),
);
@@ -1,27 +1,60 @@
import 'package:core/presentation/extensions/color_extension.dart';
import 'package:flutter/material.dart';
import 'package:tmail_ui_user/features/base/extensions/context_menu_action_list_extension.dart';
import 'package:tmail_ui_user/features/base/widget/context_menu/context_menu_dialog_item.dart';
import 'package:tmail_ui_user/features/base/widget/context_menu/context_menu_item_action.dart';
class ContextMenuDialogView extends StatelessWidget {
final List<ContextMenuItemAction> actions;
final OnContextMenuActionClick onContextMenuActionClick;
final bool useGroupedActions;
const ContextMenuDialogView({
super.key,
required this.actions,
required this.onContextMenuActionClick,
this.useGroupedActions = false,
});
@override
Widget build(BuildContext context) {
List<Widget> childrenWidget = [];
if (useGroupedActions) {
final groupedActions = actions.groupByCategory();
final entries = groupedActions.entries.toList();
childrenWidget = [
for (var i = 0; i < entries.length; i++)
...[
...entries[i].value.map((menuAction) => ContextMenuDialogItem(
key: menuAction.key != null ? Key(menuAction.key!) : null,
menuAction: menuAction,
onContextMenuActionClick: onContextMenuActionClick,
)),
if (i < entries.length - 1)
Padding(
padding: const EdgeInsets.symmetric(vertical: 8),
child: Divider(
height: 1,
color: AppColor.gray424244.withValues(alpha: 0.12),
),
),
],
];
} else {
childrenWidget = actions
.map((menuAction) => ContextMenuDialogItem(
key: menuAction.key != null ? Key(menuAction.key!) : null,
menuAction: menuAction,
onContextMenuActionClick: onContextMenuActionClick,
))
.toList();
}
return SingleChildScrollView(
child: Column(
mainAxisSize: MainAxisSize.min,
children: actions.map((menuAction) => ContextMenuDialogItem(
key: menuAction.key != null ? Key(menuAction.key!) : null,
menuAction: menuAction,
onContextMenuActionClick: onContextMenuActionClick,
)).toList(),
children: childrenWidget,
),
);
}
@@ -7,11 +7,12 @@ typedef OnContextMenuActionClick = void Function(ContextMenuItemAction action);
abstract class ContextMenuItemAction<T> with EquatableMixin {
final T action;
final String? key;
final int category;
ContextMenuItemAction(this.action, {this.key});
ContextMenuItemAction(this.action, {this.key, this.category = -1});
@override
List<Object?> get props => [action, key];
List<Object?> get props => [action, key, category];
String get actionName;
@@ -34,7 +35,7 @@ mixin OptionalSelectedIcon<T> {
abstract class ContextMenuItemActionRequiredIcon<T>
extends ContextMenuItemAction<T> with OptionalIcon {
ContextMenuItemActionRequiredIcon(super.action, {super.key});
ContextMenuItemActionRequiredIcon(super.action, {super.key, super.category});
}
abstract class ContextMenuItemActionRequiredSelectedIcon<T>
@@ -42,12 +43,17 @@ abstract class ContextMenuItemActionRequiredSelectedIcon<T>
final T? selectedAction;
ContextMenuItemActionRequiredSelectedIcon(super.action, this.selectedAction, {super.key});
ContextMenuItemActionRequiredSelectedIcon(
super.action,
this.selectedAction, {
super.key,
super.category,
});
}
abstract class ContextMenuItemActionRequiredFull<T>
extends ContextMenuItemAction<T> with OptionalIcon, OptionalSelectedIcon<T> {
final T selectedAction;
ContextMenuItemActionRequiredFull(super.action, this.selectedAction, {super.key});
ContextMenuItemActionRequiredFull(super.action, this.selectedAction, {super.key, super.category,});
}
@@ -15,6 +15,7 @@ class ContextItemEmailAction
this.appLocalizations,
this.imagePaths, {
super.key,
super.category,
});
@override
@@ -77,6 +77,7 @@ typedef OpenBottomSheetContextMenuAction = Future<void> Function({
required List<ContextMenuItemAction> itemActions,
required OnContextMenuActionClick onContextMenuActionClick,
Key? key,
bool useGroupedActions,
});
typedef OpenPopUpContextMenuAction = Future<void> Function(
@@ -592,6 +593,7 @@ class EmailActionReactor {
AppLocalizations.of(currentContext!),
imagePaths,
key: '${action.name}_action',
category: action.category,
),
)
.toList(),
@@ -599,6 +601,7 @@ class EmailActionReactor {
popBack();
handleEmailAction(presentationEmail, action.action);
},
useGroupedActions: true,
);
} else {
openPopupMenu(
@@ -9,6 +9,7 @@ extension HandleOpenContextMenuExtension on MailboxDashBoardController {
required List<ContextMenuItemAction> itemActions,
required OnContextMenuActionClick onContextMenuActionClick,
Key? key,
bool useGroupedActions = false,
}) {
if (PlatformInfo.isWeb) {
isContextMenuOpened.value = true;
@@ -18,6 +19,7 @@ extension HandleOpenContextMenuExtension on MailboxDashBoardController {
itemActions: itemActions,
onContextMenuActionClick: onContextMenuActionClick,
key: key,
useGroupedActions: useGroupedActions,
).whenComplete(() {
if (PlatformInfo.isWeb) {
isContextMenuOpened.value = false;
@@ -145,6 +145,7 @@ extension OnThreadDetailActionClick on ThreadDetailController {
action,
AppLocalizations.of(currentContext!),
imagePaths,
category: action.category,
),
)
.toList(),
@@ -152,6 +153,7 @@ extension OnThreadDetailActionClick on ThreadDetailController {
popBack();
onThreadDetailActionClick(action.action);
},
useGroupedActions: true,
);
} else {
mailboxDashBoardController.openPopupMenu(