From 8763bbf15ddf84d97251958ca58bdbf3b688da06 Mon Sep 17 00:00:00 2001 From: Christian Pauly Date: Sat, 22 Aug 2020 11:02:08 +0200 Subject: [PATCH] Fix: Status header design --- lib/views/chat.dart | 2 +- lib/views/chat_list.dart | 260 +++++++++++++++++++++------------------ 2 files changed, 140 insertions(+), 122 deletions(-) diff --git a/lib/views/chat.dart b/lib/views/chat.dart index fb939aed..929a3ccf 100644 --- a/lib/views/chat.dart +++ b/lib/views/chat.dart @@ -480,6 +480,7 @@ class _ChatState extends State<_Chat> { ), Column( children: [ + ConnectionStatusHeader(), Expanded( child: FutureBuilder( future: getTimeline(), @@ -601,7 +602,6 @@ class _ChatState extends State<_Chat> { }, ), ), - ConnectionStatusHeader(), AnimatedContainer( duration: Duration(milliseconds: 300), height: editEvent != null || replyEvent != null ? 56 : 0, diff --git a/lib/views/chat_list.dart b/lib/views/chat_list.dart index 948f77e2..404a33f8 100644 --- a/lib/views/chat_list.dart +++ b/lib/views/chat_list.dart @@ -336,130 +336,148 @@ class _ChatListState extends State { context, NewPrivateChatView()), (r) => r.isFirst), ), - body: StreamBuilder( - stream: Matrix.of(context) - .client - .onSync - .stream - .where((s) => s.hasRoomUpdate), - builder: (context, snapshot) { - return FutureBuilder( - future: waitForFirstSync(context), - builder: (BuildContext context, snapshot) { - if (snapshot.hasData) { - var rooms = List.from( - Matrix.of(context).client.rooms); - rooms.removeWhere((Room room) => - room.lastEvent == null || - (searchMode && - !room.displayname.toLowerCase().contains( - searchController.text.toLowerCase() ?? - ''))); - if (rooms.isEmpty && - (!searchMode || - publicRoomsResponse == null)) { - return Center( - child: Column( - mainAxisSize: MainAxisSize.min, - children: [ - Icon( - searchMode - ? Icons.search - : Icons.chat_bubble_outline, - size: 80, - color: Colors.grey, - ), - Text(searchMode - ? L10n.of(context).noRoomsFound - : L10n.of(context) - .startYourFirstChat), - ], - ), - ); - } - final publicRoomsCount = - (publicRoomsResponse?.chunk?.length ?? 0); - final totalCount = - rooms.length + publicRoomsCount; - final directChats = - rooms.where((r) => r.isDirectChat).toList(); - final presences = - Matrix.of(context).client.presences; - directChats.sort((a, b) => - presences[b.directChatMatrixID] - ?.presence - ?.statusMsg != - null - ? 1 - : b.lastEvent.originServerTs.compareTo( - a.lastEvent.originServerTs)); - return ListView.separated( - controller: _scrollController, - separatorBuilder: (BuildContext context, - int i) => - i == totalCount - publicRoomsCount - ? ListTile( - title: Text( - L10n.of(context).publicRooms + - ':', - style: TextStyle( - fontWeight: FontWeight.bold, - color: Theme.of(context) - .primaryColor, - ), - ), - ) - : Container(), - itemCount: totalCount + 1, - itemBuilder: (BuildContext context, int i) { - if (i == 0) { - return Column( - mainAxisSize: MainAxisSize.min, - children: [ - ConnectionStatusHeader(), - (directChats.isEmpty || - selectMode == - SelectMode.share) - ? Container() - : PreferredSize( - preferredSize: - Size.fromHeight(90), - child: Container( - height: 82, - child: ListView.builder( - scrollDirection: - Axis.horizontal, - itemCount: - directChats.length, - itemBuilder: (BuildContext - context, - int i) => - PresenceListItem( - directChats[i]), - ), - ), - ), - ], + body: Column( + children: [ + ConnectionStatusHeader(), + Expanded( + child: StreamBuilder( + stream: Matrix.of(context) + .client + .onSync + .stream + .where((s) => s.hasRoomUpdate), + builder: (context, snapshot) { + return FutureBuilder( + future: waitForFirstSync(context), + builder: (BuildContext context, snapshot) { + if (snapshot.hasData) { + var rooms = List.from( + Matrix.of(context).client.rooms); + rooms.removeWhere((Room room) => + room.lastEvent == null || + (searchMode && + !room.displayname + .toLowerCase() + .contains(searchController.text + .toLowerCase() ?? + ''))); + if (rooms.isEmpty && + (!searchMode || + publicRoomsResponse == null)) { + return Center( + child: Column( + mainAxisSize: MainAxisSize.min, + children: [ + Icon( + searchMode + ? Icons.search + : Icons.chat_bubble_outline, + size: 80, + color: Colors.grey, + ), + Text(searchMode + ? L10n.of(context).noRoomsFound + : L10n.of(context) + .startYourFirstChat), + ], + ), ); } - i--; - return i < rooms.length - ? ChatListItem( - rooms[i], - activeChat: widget.activeChat == - rooms[i].id, - ) - : PublicRoomListItem(publicRoomsResponse - .chunk[i - rooms.length]); - }); - } else { - return Center( - child: CircularProgressIndicator(), + final publicRoomsCount = + (publicRoomsResponse?.chunk?.length ?? + 0); + final totalCount = + rooms.length + publicRoomsCount; + final directChats = rooms + .where((r) => r.isDirectChat) + .toList(); + final presences = + Matrix.of(context).client.presences; + directChats.sort((a, b) => presences[ + b.directChatMatrixID] + ?.presence + ?.statusMsg != + null + ? 1 + : b.lastEvent.originServerTs.compareTo( + a.lastEvent.originServerTs)); + return ListView.separated( + controller: _scrollController, + separatorBuilder: (BuildContext context, + int i) => + i == totalCount - publicRoomsCount + ? ListTile( + title: Text( + L10n.of(context) + .publicRooms + + ':', + style: TextStyle( + fontWeight: + FontWeight.bold, + color: Theme.of(context) + .primaryColor, + ), + ), + ) + : Container(), + itemCount: totalCount + 1, + itemBuilder: + (BuildContext context, int i) { + if (i == 0) { + return Column( + mainAxisSize: MainAxisSize.min, + children: [ + (directChats.isEmpty || + selectMode == + SelectMode.share) + ? Container() + : PreferredSize( + preferredSize: + Size.fromHeight(90), + child: Container( + height: 82, + child: + ListView.builder( + scrollDirection: + Axis.horizontal, + itemCount: + directChats + .length, + itemBuilder: (BuildContext + context, + int i) => + PresenceListItem( + directChats[ + i]), + ), + ), + ), + ], + ); + } + i--; + return i < rooms.length + ? ChatListItem( + rooms[i], + activeChat: + widget.activeChat == + rooms[i].id, + ) + : PublicRoomListItem( + publicRoomsResponse + .chunk[i - rooms.length]); + }); + } else { + return Center( + child: CircularProgressIndicator(), + ); + } + }, ); - } - }, - ); - }), + }), + ), + ], + ), ); }); });