2ndbeam/fluffychat-fork
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor: Improve performance of chat list

Browse source
This commit is contained in:
krille-chan 2024-05-10 16:32:14 +02:00
commit 3c85786c10
No known key found for this signature in database
2 changed files with 24 additions and 31 deletions
Download patch file Download diff file Expand all files Collapse all files

49
lib/pages/chat_list/chat_list_body.dart
View file

@ -14,7 +14,6 @@ import 'package:fluffychat/pages/chat_list/status_msg_list.dart';
import 'package:fluffychat/pages/chat_list/utils/on_chat_tap.dart'; import 'package:fluffychat/pages/chat_list/utils/on_chat_tap.dart';
import 'package:fluffychat/pages/user_bottom_sheet/user_bottom_sheet.dart'; import 'package:fluffychat/pages/user_bottom_sheet/user_bottom_sheet.dart';
import 'package:fluffychat/utils/adaptive_bottom_sheet.dart'; import 'package:fluffychat/utils/adaptive_bottom_sheet.dart';
import 'package:fluffychat/utils/matrix_sdk_extensions/matrix_locals.dart';
import 'package:fluffychat/utils/stream_extension.dart'; import 'package:fluffychat/utils/stream_extension.dart';
import 'package:fluffychat/widgets/avatar.dart'; import 'package:fluffychat/widgets/avatar.dart';
import 'package:fluffychat/widgets/public_room_bottom_sheet.dart'; import 'package:fluffychat/widgets/public_room_bottom_sheet.dart';
@ -43,7 +42,7 @@ class ChatListViewBody extends StatelessWidget {
Theme.of(context).textTheme.bodyLarge!.color!.withAlpha(100); Theme.of(context).textTheme.bodyLarge!.color!.withAlpha(100);
final subtitleColor = final subtitleColor =
Theme.of(context).textTheme.bodyLarge!.color!.withAlpha(50); Theme.of(context).textTheme.bodyLarge!.color!.withAlpha(50);
final filter = controller.searchController.text.toLowerCase();
return PageTransitionSwitcher( return PageTransitionSwitcher(
transitionBuilder: ( transitionBuilder: (
Widget child, Widget child,
@ -239,35 +238,23 @@ class ChatListViewBody extends StatelessWidget {
), ),
), ),
if (client.prevBatch != null) if (client.prevBatch != null)
SliverList( SliverList.builder(
delegate: SliverChildBuilderDelegate( itemCount: rooms.length,
(BuildContext context, int i) { itemBuilder: (BuildContext context, int i) {
if (!rooms[i] return ChatListItem(
.getLocalizedDisplayname( rooms[i],
MatrixLocals(L10n.of(context)!), key: Key('chat_list_item_${rooms[i].id}'),
) filter: filter,
.toLowerCase() selected:
.contains( controller.selectedRoomIds.contains(rooms[i].id),
controller.searchController.text.toLowerCase(), onTap: controller.selectMode == SelectMode.select
)) { ? () => controller.toggleSelection(rooms[i].id)
return const SizedBox.shrink(); : () => onChatTap(rooms[i], context),
} onLongPress: () =>
final activeChat = controller.activeChat == rooms[i].id; controller.toggleSelection(rooms[i].id),
return ChatListItem( activeChat: controller.activeChat == rooms[i].id,
rooms[i], );
key: Key('chat_list_item_${rooms[i].id}'), },
selected:
controller.selectedRoomIds.contains(rooms[i].id),
onTap: controller.selectMode == SelectMode.select
? () => controller.toggleSelection(rooms[i].id)
: () => onChatTap(rooms[i], context),
onLongPress: () =>
controller.toggleSelection(rooms[i].id),
activeChat: activeChat,
);
},
childCount: rooms.length,
),
), ),
], ],
), ),

6
lib/pages/chat_list/chat_list_item.dart
View file

@ -22,6 +22,7 @@ class ChatListItem extends StatelessWidget {
final void Function()? onLongPress; final void Function()? onLongPress;
final void Function()? onForget; final void Function()? onForget;
final void Function() onTap; final void Function() onTap;
final String? filter;
const ChatListItem( const ChatListItem(
this.room, { this.room, {
@ -30,6 +31,7 @@ class ChatListItem extends StatelessWidget {
required this.onTap, required this.onTap,
this.onLongPress, this.onLongPress,
this.onForget, this.onForget,
this.filter,
super.key, super.key,
}); });
@ -83,6 +85,10 @@ class ChatListItem extends StatelessWidget {
final displayname = room.getLocalizedDisplayname( final displayname = room.getLocalizedDisplayname(
MatrixLocals(L10n.of(context)!), MatrixLocals(L10n.of(context)!),
); );
final filter = this.filter;
if (filter != null && !displayname.toLowerCase().contains(filter)) {
return const SizedBox.shrink();
}
return Padding( return Padding(
padding: const EdgeInsets.symmetric( padding: const EdgeInsets.symmetric(
horizontal: 8, horizontal: 8,