Merge branch 'develop' into feat/reverseMessageListView

This commit is contained in:
Sahil Kumar
2021-07-22 15:35:40 +05:30
committed by GitHub
7 changed files with 54 additions and 14 deletions
@@ -2,6 +2,7 @@
✅ Added
- Added `MessageListView.paginationLimit`
- Allow the various ListView widgets to be themed via ThemeData classes
🐞 Fixed
@@ -167,6 +167,7 @@ class MessageListView extends StatefulWidget {
this.threadSeparatorBuilder,
this.messageListController,
this.reverse = true,
this.paginationLimit = 20,
}) : super(key: key);
/// Function used to build a custom message widget
@@ -178,6 +179,9 @@ class MessageListView extends StatefulWidget {
///
/// See [ScrollView.reverse].
final bool reverse;
/// Limit used during pagination
final int paginationLimit;
/// Function used to build a custom system message widget
final SystemMessageBuilder? systemMessageBuilder;
@@ -336,6 +340,7 @@ class _MessageListViewState extends State<MessageListView> {
@override
Widget build(BuildContext context) => MessageListCore(
paginationLimit: widget.paginationLimit,
messageFilter: widget.messageFilter,
loadingBuilder: widget.loadingBuilder ??
(context) => const Center(
@@ -664,7 +669,9 @@ class _MessageListViewState extends State<MessageListView> {
);
Future<void> _paginateData(
StreamChannelState? channel, QueryDirection direction) =>
StreamChannelState? channel,
QueryDirection direction,
) =>
_messageListController.paginateData!(direction: direction);
int? _getTopElementIndex(Iterable<ItemPosition> values) {
+1 -1
View File
@@ -30,7 +30,7 @@ dependencies:
lottie: ^1.0.1
meta: ^1.3.0
path_provider: ^2.0.1
photo_manager: ^1.1.6
photo_manager: ^1.2.6+1
photo_view: ^0.12.0
rxdart: ^0.27.0
scrollable_positioned_list: ^0.2.0-nullsafety.0
@@ -1,3 +1,8 @@
## Upcoming
✅ Added
- Added `MessageListCore.paginationLimit`
## 2.0.0
🛑️ Breaking Changes from `1.5.3`
@@ -71,6 +71,7 @@ class MessageListCore extends StatefulWidget {
this.parentMessage,
this.messageListController,
this.messageFilter,
this.paginationLimit = 20,
}) : super(key: key);
/// A [MessageListController] allows pagination.
@@ -86,6 +87,9 @@ class MessageListCore extends StatefulWidget {
/// Function used to build an empty widget
final WidgetBuilder emptyBuilder;
/// Limit used to paginate messages
final int paginationLimit;
/// Callback triggered when an error occurs while performing the given
/// request.
///
@@ -163,13 +167,20 @@ class MessageListCoreState extends State<MessageListCore> {
/// Fetches more messages with updated pagination and updates the widget.
///
/// Optionally pass the fetch direction, defaults to [QueryDirection.top]
/// Optionally pass a limit, defaults to 20
Future<void> paginateData({
QueryDirection direction = QueryDirection.top,
}) {
if (!_isThreadConversation) {
return _streamChannel!.queryMessages(direction: direction);
return _streamChannel!.queryMessages(
direction: direction,
limit: widget.paginationLimit,
);
} else {
return _streamChannel!.getReplies(widget.parentMessage!.id);
return _streamChannel!.getReplies(
widget.parentMessage!.id,
limit: widget.paginationLimit,
);
}
}
@@ -179,7 +190,10 @@ class MessageListCoreState extends State<MessageListCore> {
if (newStreamChannel != _streamChannel) {
if (_streamChannel == null /*only first time*/ && _isThreadConversation) {
newStreamChannel.getReplies(widget.parentMessage!.id);
newStreamChannel.getReplies(
widget.parentMessage!.id,
limit: widget.paginationLimit,
);
}
_streamChannel = newStreamChannel;
}
@@ -197,7 +211,10 @@ class MessageListCoreState extends State<MessageListCore> {
if (widget.parentMessage?.id != widget.parentMessage?.id) {
if (_isThreadConversation) {
_streamChannel!.getReplies(widget.parentMessage!.id);
_streamChannel!.getReplies(
widget.parentMessage!.id,
limit: widget.paginationLimit,
);
}
}
}
@@ -147,9 +147,18 @@ class StreamChannelState extends State<StreamChannel> {
}
/// Calls [channel.query] updating [queryMessage] stream
Future<void> queryMessages({QueryDirection? direction = QueryDirection.top}) {
if (direction == QueryDirection.top) return _queryTopMessages();
return _queryBottomMessages();
Future<void> queryMessages({
QueryDirection? direction = QueryDirection.top,
int limit = 20,
}) {
if (direction == QueryDirection.top) {
return _queryTopMessages(
limit: limit,
);
}
return _queryBottomMessages(
limit: limit,
);
}
/// Calls [channel.getReplies] updating [queryMessage] stream
@@ -151,7 +151,9 @@ void main() {
(tester) async {
const messageListCoreKey = Key('messageListCore');
final controller = MessageListController();
const paginationLimit = 10;
final messageListCore = MessageListCore(
paginationLimit: paginationLimit,
key: messageListCoreKey,
messageListBuilder: (_, __) => const Offstage(),
loadingBuilder: (BuildContext context) => const Offstage(),
@@ -165,10 +167,6 @@ void main() {
final mockChannel = MockChannel();
when(() => mockChannel.state.isUpToDate).thenReturn(true);
// when(() => mockChannel.query(
// messagesPagination: any(named: 'messagesPagination'),
// preferOffline: any(named: 'preferOffline'),
// )).thenAnswer((_) => mockChannel.state);
final messages = _generateMessages();
when(() => mockChannel.state.messages).thenReturn(messages);
when(() => mockChannel.state.messagesStream)
@@ -191,7 +189,10 @@ void main() {
await coreState.paginateData();
verify(() => mockChannel.query(
messagesPagination: any(named: 'messagesPagination'),
messagesPagination: any(
named: 'messagesPagination',
that: wrapMatcher((it) => it.limit == paginationLimit),
),
preferOffline: any(named: 'preferOffline'),
)).called(1);
},